Permits Table Schema - Githubissues

0x4007 commented 11 months ago

Permits

Holds the permit details associated with some credits.

Field	Type	Description	Example Value
`id`	int8	Unique identifier for the permit entry.	101
`created`	timestamptz	Timestamp when the permit was created.	2023-09-15T11:20:23.004Z
`updated`	timestamptz	Timestamp when the permit was last updated.	2023-09-15T11:22:40.050Z
`network`	smallserial	Network identifier.	100
`token`	CHAR(42)	Token associated with the permit.	0xe92E133E0b41519A2Ce9Dd4D7945Fa863463a97e
`amount`	int8	Amount associated with the token for the permit.	150000000000000000000
`nonce`	int8	Nonce associated with the permit.	8802044552...
`deadline`	int8	Deadline associated with the permit.	115792089237316195423570985008687907853269984665640564039457584007913129640000
`beneficiary`	CHAR(42)	Destination address for the permit's transfer.	0x4008FE2190c8F3E18098aeB4A40bb8eC149a342d
`owner`	CHAR(42)	Owner of the credit.	0xf88da4592C791213e52840d120E7E0B38B818bD2
`signature`	CHAR(132)	Signature associated with the credit.	0xecb65ed54b2ee7a0ffad5aff8314eef684f80dcb78c18b4ea9f588e083a0fa114546d466befa23cb81d040957615b5b1ad08966b64c5778802e5e115cdcd39241b

seprintour commented 11 months ago

/start

ubiquibot[bot] commented 11 months ago





Deadline
Sat, 23 Sep 2023 17:37:41 UTC


Registered Wallet
0x3623338046b101ecEc741De9C3594CC2176f39E5

Tips:

Use /wallet 0x0000...0000 if you want to update your registered payment wallet address @user.
Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the bounty.

seprintour commented 11 months ago

Screenshot 2023-09-23 at 4 57 02 PM

@pavlovcik deadline is out of range for type int8

I am trying to move existing data from permit to the new table, so the number is from one of the permit data that is apparently out of range for it

0x4007 commented 11 months ago

/ask what data type should we use

ubiquibot[bot] commented 11 months ago

! Error: This model's maximum context length is 16385 tokens. However, you requested 17805 tokens (1421 in the messages, 16384 in the completion). Please reduce the length of the messages or completion.

0x4007 commented 11 months ago

Oh I just realized something. This is a config error I think.

0x4007 commented 11 months ago

/ask what data type should we use

0x4007 commented 11 months ago

@pavlovcik deadline is out of range for type int8

I am trying to move existing data from permit to the new table, so the number is from one of the permit data that is apparently out of range for it

Chatgpt also recommended timestamptz but unsure if that's valid. I have to check in a bit I'm a bit occupied at the moment

seprintour commented 11 months ago

This is the deadline 115792089237316195423570985008687907853269984665640564039457584007913129639935

seprintour commented 11 months ago

Had to make deadline, nonce and amount numeric, bigint or int8 did not cut it

seprintour commented 11 months ago

Had to make deadline, nonce and amount numeric, bigint or int8 did not cut it

This means i'd have to change the Debit amount to numeric too, int8 supports up to 1e18 but no permit amount is just going to be 1 ETH or equivalent.

So it goes past 18 and int8 returns an out of range error

seprintour commented 11 months ago

Screenshot 2023-09-23 at 5 39 39 PM

0x4007 commented 11 months ago

Another idea I might add to a new scope, but we can save so much on memory if we have:

a foreign key to recognized partner wallet address (partner_wallet_id)
recognized token addresses (token_id) and a new tokens table.
bounty hunter address should be a foreign key also (wallet_id).

Tokens table:

id, created, updated, network, address

ubiquibot[bot] commented 10 months ago

Do you have any updates @seprintour? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Wed Oct 04 2023 02:12:08 GMT+0000 (Coordinated Universal Time)

seprintour commented 10 months ago

Still in review

ubiquibot[bot] commented 10 months ago

Do you have any updates @seprintour? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Mon Oct 09 2023 11:32:12 GMT+0000 (Coordinated Universal Time)

seprintour commented 10 months ago

Going to test the new implementations

ubiquibot[bot] commented 10 months ago

Do you have any updates @seprintour? If you would like to release the bounty back to the DevPool, please comment /stop Last activity time: Sun Oct 15 2023 23:28:05 GMT+0000 (Coordinated Universal Time)

ubiquibot[bot] commented 10 months ago

@seprintour - Releasing the bounty back to dev pool because the allocated duration already ended! Last activity time: Sun Oct 15 2023 23:28:05 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-17T12:34:24.911Z

ubiquibot[bot] commented 6 months ago

+ Evaluating results. Please wait...

ubiquibot[bot] commented 6 months ago

[ 415.8 WXDAI ]

@pavlovcik

Contributions Overview

View	Contribution	Count	Reward
Issue	Specification	1	80
Issue	Task	1.00	112.5
Issue	Comment	3	33.4
Issue	Comment	3	0
Review	Comment	15	126.6
Review	Comment	15	63.3

Conversation Incentives

Comment	Formatting	Relevance	Reward
## Permits Holds the permit details associated with some ...	80 h2: count: 1 score: "1" words: 3 code: count: 11 score: "11" words: 11 td: count: 44 score: "44" words: 129	1	80
Oh I just realized something. This is a config error I think. ...	2.4	0.52	2.4
> ![Screenshot 2023-09-23 at 4 57 02 PM](https://github.com/ubiq...	7 code: count: 2 score: "2" words: 2	0.81	7
Another idea I might add to a new scope, but we can save so much...	24 li: count: 3 score: "3" words: 36 code: count: 4 score: "4" words: 4 hr: count: 1 score: "1" words: 0	0.44	24
Oh I just realized something. This is a config error I think. ...	-	0.52	-
> ![Screenshot 2023-09-23 at 4 57 02 PM](https://github.com/ubiq...	- code: count: 2 score: "0" words: 2	0.81	-
Another idea I might add to a new scope, but we can save so much...	- li: count: 3 score: "0" words: 36 code: count: 4 score: "0" words: 4 hr: count: 1 score: "0" words: 0	0.44	-
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
> The bot is expected to be stable after these changes are imple...	2.4	-	2.4
Will need to e2e test and merge...	1.4	-	1.4
To be honest I spent a significant amount of time trying to comp...	13.2	-	13.2
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
> The bot is expected to be stable after these changes are imple...	1.2	-	1.2
Will need to e2e test and merge...	0.7	-	0.7
To be honest I spent a significant amount of time trying to comp...	6.6	-	6.6
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

[ 21.45 WXDAI ]

@seprintour

Contributions Overview

View	Contribution	Count	Reward
Issue	Comment	7	14.8
Review	Comment	8	6.65

Conversation Incentives

Comment	Formatting	Relevance	Reward
![Screenshot 2023-09-23 at 4 57 02 PM](https://github.com/ubiqui...	4.5 code: count: 2 score: "0.5" words: 2	0.75	4.5
This is the deadline `115792089237316195423570985008687907853269...	0.75 code: count: 1 score: "0.25" words: 1	0.73	0.75
Had to make `deadline`, `nonce` and `amount` numeric, bigint or ...	2.25 code: count: 3 score: "0.75" words: 3	0.73	2.25
> Had to make `deadline`, `nonce` and `amount` numeric, bigint o...	6.4 code: count: 8 score: "2" words: 9	0.555	6.4
![Screenshot 2023-09-23 at 5 39 39 PM](https://github.com/ubiqui...	-	0.375	-
Still in review ...	0.3	0.53	0.3
Going to test the new implementations...	0.6	0.41	0.6
@pavlovcik PR attached...	0.3	-	0.3
> From #787 > > Shouldn't you refactor functions that interact ...	0.6	-	0.6
> Shouldn't you refactor functions that interact with these tabl...	1.4	-	1.4
> Code looks good just have some rename requests Fixed issues...	0.2	-	0.2
Ping @pavlovcik @whilefoo ...	0.3	-	0.3
> Even if we ignore the bug with char(x), people are mostly reco...	2.2	-	2.2
@0xcodercrane @whilefoo i made it a `text` with a length constra...	1.55 code: count: 1 score: "0.25" words: 1	-	1.55
> @seprintour can you make new QA issue to make sure everything ...	0.1	-	0.1

[ 0 WXDAI ]

@BeanieMen

Contributions Overview

View	Contribution	Count	Reward
Review	Comment	1	0

Conversation Incentives

Comment	Formatting	Relevance	Reward
🎉...	-	-	-

[ 52.6 WXDAI ]

@gitcoindev

Contributions Overview

View	Contribution	Count	Reward
Review	Comment	7	52.6

Conversation Incentives

Comment	Formatting	Relevance	Reward
> 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

[ 1.6 WXDAI ]

@0xcodercrane

Contributions Overview

View	Contribution	Count	Reward
Review	Comment	1	1.6

Conversation Incentives

Comment	Formatting	Relevance	Reward
should I merge https://github.com/ubiquity/ubiquibot/pull/643 in...	1.6	-	1.6

[ 0.6 WXDAI ]

@wannacfuture

Contributions Overview

View	Contribution	Count	Reward
Review	Comment	1	0.6

Conversation Incentives

Comment	Formatting	Relevance	Reward
should I resolve the conflicts? @pavlovcik ...	0.6	-	0.6

[ 139.7 WXDAI ]

@rndquu

Contributions Overview

View	Contribution	Count	Reward
Review	Comment	3	139.7

Conversation Incentives

Comment	Formatting	Relevance	Reward
@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

Comment

Formatting

Relevance

Reward

@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

[ 26.7 WXDAI ]

@whilefoo

Contributions Overview

View	Contribution	Count	Reward
Review	Comment	6	26.7

Conversation Incentives

Comment	Formatting	Relevance	Reward
@seprintour can you make new QA issue to make sure everything is...	1.3	-	1.3
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

View	Contribution	Count	Reward
Review	Comment	5	3.7

Conversation Incentives

Comment	Formatting	Relevance	Reward
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

Deadline	Sat, 23 Sep 2023 17:37:41 UTC
Registered Wallet	0x3623338046b101ecEc741De9C3594CC2176f39E5

ubiquity / ubiquibot

Permits Table Schema #811

Permits

Tips:

@pavlovcik

Contributions Overview

Conversation Incentives

@seprintour

Contributions Overview

Conversation Incentives

@BeanieMen

Contributions Overview

Conversation Incentives

@gitcoindev

Contributions Overview

Conversation Incentives

@0xcodercrane

Contributions Overview

Conversation Incentives

@wannacfuture

Contributions Overview

Conversation Incentives

@rndquu

Contributions Overview

Conversation Incentives

@whilefoo

Contributions Overview

Conversation Incentives

@EtherealGlow

Contributions Overview

Conversation Incentives