ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
17 stars 59 forks source link

Database Re-Architecture #812

Closed 0x4007 closed 6 months ago

0x4007 commented 11 months ago

Tables

Access

The columns should be renamed and adhere to the above rules.

id, created, updated, user_id, organization_id, repository_id, multiplier, priority, time, price.

Issues

Should be deleted. No idea why this exists in the first place. @0xcodercrane rfc

This seems to be constantly updated so its logic must be disconnected from the bot side.

Logs

Will make an exception since they are ephemeral and connected to multiple systems.

Multiplier

Delete.

Penalty

Permits

Users

Seems nice to have even if we have no immediate plans for the data. We will leave it unchanged.

Wallets

id, created, updated, user_id, wallet (char42)

Keyrxng commented 11 months ago

/start

ubiquibot[bot] commented 11 months ago

Too many assigned issues, you have reached your max of 2

molecula451 commented 11 months ago

/start

ubiquibot[bot] commented 11 months ago

Deadline Sun, 24 Sep 2023 01:41:21 UTC
Registered Wallet 0x4D0704f400D57Ba93eEa88765C3FcDBD826dCFc4
Tips:
ubiquibot[bot] commented 11 months ago

Do you have any updates @molecula451? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Sat Sep 23 2023 21:41:14 GMT+0000 (Coordinated Universal Time)

ubiquibot[bot] commented 11 months ago

Do you have any updates @molecula451? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Sat Sep 23 2023 21:41:14 GMT+0000 (Coordinated Universal Time)

molecula451 commented 10 months ago

/start

ubiquibot[bot] commented 10 months ago

Deadline Tue, 03 Oct 2023 11:22:03 UTC
Registered Wallet 0x4D0704f400D57Ba93eEa88765C3FcDBD826dCFc4

Tips:

ubiquibot[bot] commented 10 months ago

Do you have any updates @molecula451? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Tue Oct 03 2023 07:21:57 GMT+0000 (Coordinated Universal Time)

ubiquibot[bot] commented 10 months ago

@molecula451 - Releasing the bounty back to dev pool because the allocated duration already ended! Last activity time: Tue Oct 03 2023 07:21:57 GMT+0000 (Coordinated Universal Time)

molecula451 commented 10 months ago

/start

ubiquibot[bot] commented 10 months ago

Deadline Wed, 11 Oct 2023 01:59:51 UTC
Registered Wallet 0x4D0704f400D57Ba93eEa88765C3FcDBD826dCFc4

Tips:

0xcodercrane commented 10 months ago

Should be deleted. No idea why this exists in the first place. @0xcodercrane rfc

Originally created it for data analytics of issues. but looks like no need to use it for even weekly reports. agree to delete. @molecula451

ubiquibot[bot] commented 10 months ago

Do you have any updates @molecula451? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Tue Oct 10 2023 21:59:47 GMT+0000 (Coordinated Universal Time)

ubiquibot[bot] commented 10 months ago

@molecula451 - Releasing the bounty back to dev pool because the allocated duration already ended! Last activity time: Tue Oct 10 2023 21:59:47 GMT+0000 (Coordinated Universal Time)

ubiquibot[bot] commented 10 months ago

@pavlovcik The time limit for this bounty is on Thu, 19 Oct 2023 08:45:31 UTC

ubiquibot[bot] commented 10 months ago

@pavlovcik - Releasing the bounty back to dev pool because the allocated duration already ended! Last activity time: Sat Sep 23 2023 13:50:11 GMT+0000 (Coordinated Universal Time)

molecula451 commented 10 months ago

/start

ubiquibot[bot] commented 10 months ago

Deadline Sun, 22 Oct 2023 18:12:08 UTC
Registered Wallet 0x4D0704f400D57Ba93eEa88765C3FcDBD826dCFc4

Tips:

rndquu commented 10 months ago

/start

I don't think the current issue is relevant/available because (as far as I understand) it's going to be solved as a part of https://github.com/ubiquity/ubiquibot/pull/644

molecula451 commented 10 months ago

i was thinking the same after seeing #644 linked to this issue but i didnt see relevant/related commits at #644

molecula451 commented 10 months ago

@pavlovcik can you confirm us?

Also can you confirm this?

multi

on "Access" table you are mentioning "multiplier" but you also mention Multiplier to be delete

0x4007 commented 10 months ago

I consolidated multiplier details in the access table. Perhaps we should rename the access table but it holds the miscellaneous user information.

I guess I don't get "activity credit" cause I'm active on a branch off of my linked pull request right now, rebuilding the comment incentives code.

molecula451 commented 10 months ago

@pavlovcik look what i have for you

https://github.com/ubiquity/ubiquibot/assets/41552663/5347a448-7d92-482c-b106-e7d77c43fc72

the 'wallet' table is ready

@rndquu

rndquu commented 10 months ago

@pavlovcik look what i have for you

id.mp4 the 'wallet' table is ready

@rndquu

This is great but I still think that the current issue should be solved after https://github.com/ubiquity/ubiquibot/pull/644 is merged because https://github.com/ubiquity/ubiquibot/issues/812 and https://github.com/ubiquity/ubiquibot/pull/644 are heavily interconnected

molecula451 commented 10 months ago

well yes, makes a bit of some sense just in case the bot unassign me and the /start command it's still disabled this is the branch i was working https://github.com/ubiquity/ubiquibot/compare/development...molecula451:ubiquibot:db-testing these are the changes for the table so it's hopefully useful to pavlovick

ubiquibot[bot] commented 10 months ago

Do you have any updates @molecula451? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Thu Oct 26 2023 19:46:02 GMT+0000 (Coordinated Universal Time)

ubiquibot[bot] commented 9 months ago

@molecula451 - Releasing the bounty back to dev pool because the allocated duration already ended! Last activity time: Thu Oct 26 2023 19:46:02 GMT+0000 (Coordinated Universal Time)

ubiquibot[bot] commented 6 months ago
+ Evaluating results. Please wait...
ubiquibot[bot] commented 6 months ago

@pavlovcik the deadline is at 2024-02-17T14:33:46.314Z

ubiquibot[bot] commented 6 months ago
+ Evaluating results. Please wait...
ubiquibot[bot] commented 6 months ago

[ 458.3 WXDAI ]

@pavlovcik
Contributions Overview
ViewContributionCountReward
IssueSpecification156.8
IssueTask1.00225
IssueComment110
IssueComment10
ReviewComment13111
ReviewComment1355.5
Conversation Incentives
CommentFormattingRelevanceReward
# Tables - Every table should start with the following column...
56.8
h1:
  count: 1
  score: "1"
  words: 1
h2:
  count: 8
  score: "8"
  words: 8
li:
  count: 5
  score: "5"
  words: 98
code:
  count: 5
  score: "5"
  words: 4
156.8
I consolidated multiplier details in the access table. Perhaps w...
100.610
I consolidated multiplier details in the access table. Perhaps w...
-0.6-
This should have merged this in weeks ago. The number of outstan...
18.2-18.2
@0xcodercrane In this case you should consider just regenerating...
7.6-7.6
Don't think it matters the order regarding conflicts but technic...
3.8-3.8
Will need to e2e test and merge...
1.4-1.4
Review comments at https://github.com/ubiquity/ubiquibot/pull/84...
2-2
There are many dimensions of your proposal to consider so it's h...
16.8-16.8
This should work on personal accounts (not just organizations) a...
4.8-4.8
Notes to self on incentive calculation architecture: - work bac...
9.6
li:
  count: 2
  score: "4"
  words: 21
-9.6
I think next week. It feels really close to being done but I'm a...
5.6-5.6
@wannacfuture its better that you branch from here instead of pu...
3.8-3.8
> I'm trying to use the new version with workflow dispatcher and...
17
code:
  count: 1
  score: "2"
  words: 26
-17
You guys should make sure you're using the same version of Node....
18.4-18.4
@gitcoindev maybe you can help fix the knip CI here...
2-2
This should have merged this in weeks ago. The number of outstan...
9.1-9.1
@0xcodercrane In this case you should consider just regenerating...
3.8-3.8
Don't think it matters the order regarding conflicts but technic...
1.9-1.9
Will need to e2e test and merge...
0.7-0.7
Review comments at https://github.com/ubiquity/ubiquibot/pull/84...
1-1
There are many dimensions of your proposal to consider so it's h...
8.4-8.4
This should work on personal accounts (not just organizations) a...
2.4-2.4
Notes to self on incentive calculation architecture: - work bac...
4.8
li:
  count: 2
  score: "2"
  words: 21
-4.8
I think next week. It feels really close to being done but I'm a...
2.8-2.8
@wannacfuture its better that you branch from here instead of pu...
1.9-1.9
> I'm trying to use the new version with workflow dispatcher and...
8.5
code:
  count: 1
  score: "1"
  words: 26
-8.5
You guys should make sure you're using the same version of Node....
9.2-9.2
@gitcoindev maybe you can help fix the knip CI here...
1-1

[ 12.6 WXDAI ]

@molecula451
Contributions Overview
ViewContributionCountReward
IssueComment412.6
Conversation Incentives
CommentFormattingRelevanceReward
i was thinking the same after seeing #644 linked to this issue b...
2.10.432.1
@pavlovcik can you confirm us? Also can you confirm this? ...
2.50.682.5
@pavlovcik look what i have for you https://github.com/ubiqui...
2.50.692.5
well yes, makes a bit of some sense just in case the bot unassig...
5.50.675.5

[ 0 WXDAI ]

@BeanieMen
Contributions Overview
ViewContributionCountReward
ReviewComment10
Conversation Incentives
CommentFormattingRelevanceReward
šŸŽ‰...
---

[ 52.6 WXDAI ]

@gitcoindev
Contributions Overview
ViewContributionCountReward
ReviewComment752.6
Conversation Incentives
CommentFormattingRelevanceReward
> There also seems to be an issue with ubiquibot-logger > > `...
9.1
code:
  count: 4
  score: "4"
  words: 4
-9.1
> > Hi @whilefoo , the `ubiquibot-logger` is an ESM module https...
7.3
code:
  count: 5
  score: "5"
  words: 7
-7.3
> > > Hi @whilefoo , the `ubiquibot-logger` is an ESM module htt...
14.9
code:
  count: 7
  score: "7"
  words: 10
-14.9
hi @whilefoo I will give this priority and try to fix till Monda...
1.4-1.4
@whilefoo I am on v0.3.4 already, so far no luck but closer and ...
5.2-5.2
hi @whilefoo could you please check ubiquibot-logger v0.3.5 ? I...
8.9
code:
  count: 1
  score: "1"
  words: 6
-8.9
Btw, about @pavlovcik 's last comment, I agree that we should sy...
5.8-5.8

[ 4 WXDAI ]

@0xcodercrane
Contributions Overview
ViewContributionCountReward
IssueComment12.4
ReviewComment11.6
Conversation Incentives
CommentFormattingRelevanceReward
> Should be deleted. No idea why this exists in the first place....
2.40.772.4
should I merge https://github.com/ubiquity/ubiquibot/pull/643 in...
1.6-1.6

[ 0.6 WXDAI ]

@wannacfuture
Contributions Overview
ViewContributionCountReward
ReviewComment10.6
Conversation Incentives
CommentFormattingRelevanceReward
should I resolve the conflicts? @pavlovcik ...
0.6-0.6

[ 147.3 WXDAI ]

@rndquu
Contributions Overview
ViewContributionCountReward
IssueComment27.6
ReviewComment3139.7
Conversation Incentives
CommentFormattingRelevanceReward
> /start I don't think the current issue is relevant/availabl...
3.30.533.3
> @pavlovcik look what i have for you > > id.mp4 > the 'wa...
4.30.554.3
@pavlovcik There is the supabase [diff tool](https://supabas...
42.1
a:
  count: 4
  score: "4"
  words: 7
li:
  count: 7
  score: "7"
  words: 110
code:
  count: 10
  score: "10"
  words: 11
-42.1
@pavlovcik Regarding the `locations` and metadata. This i...
96
h2:
  count: 2
  score: "2"
  words: 11
h3:
  count: 4
  score: "4"
  words: 4
a:
  count: 2
  score: "2"
  words: 3
li:
  count: 10
  score: "10"
  words: 170
code:
  count: 21
  score: "21"
  words: 67
td:
  count: 16
  score: "16"
  words: 57
-96
> Alternatively we do the two API calls but there's a good chanc...
1.6-1.6

[ 25.4 WXDAI ]

@whilefoo
Contributions Overview
ViewContributionCountReward
ReviewComment525.4
Conversation Incentives
CommentFormattingRelevanceReward
I'm trying to use the new version with workflow dispatcher and I...
6.8
code:
  count: 1
  score: "1"
  words: 26
-6.8
There also seems to be an issue with ubiquibot-logger ``` co...
7.9
code:
  count: 1
  score: "1"
  words: 0
-7.9
> Hi @whilefoo , the `ubiquibot-logger` is an ESM module https:/...
8.9
code:
  count: 5
  score: "5"
  words: 7
-8.9
> Following the article https://dev.to/tigawanna/building-and-pu...
1.2-1.2
@gitcoindev it's working now, thanks!...
0.6-0.6

[ 3.7 WXDAI ]

@EtherealGlow
Contributions Overview
ViewContributionCountReward
ReviewComment53.7
Conversation Incentives
CommentFormattingRelevanceReward
I pray for the refactor to be done quickly šŸ™...
0.9-0.9
Do you guys have any estimate on when the refactor will be done ...
1.3-1.3
Any new estimate for how much time is left?...
0.9-0.9
šŸ™...
---
When will the refactor be done šŸ˜­...
0.6-0.6