-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Payload optimization #19
Comments
|
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Specification | 1 | 9.18 |
Review | Comment | 10 | 14.633 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
This is an example of a diff q&a gone bad. Too much was sent… | 3.06content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 56 wordValue: 0.1 result: 3.06 | 1 | 9.18 |
Perhaps my spec isnt clear but youre not following it. - Remove… | 5.97content: content: p: score: 0 elementCount: 1 ul: score: 1 elementCount: 1 li: score: 0.5 elementCount: 4 result: 3 regex: wordCount: 54 wordValue: 0.1 result: 2.97 | 0.9 | 5.673 |
Should definitely be dynamic from git ignore and such. Otherwise… | 1.11content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 17 wordValue: 0.1 result: 1.11 | 0.8 | 0.888 |
Wrong approach. Should not be any hard cut off from some arbitra… | 1.06content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 16 wordValue: 0.1 result: 1.06 | 0.7 | 0.742 |
Hey i just realized this is pointless. If its ignored it wont b… | 2.2content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 38 wordValue: 0.1 result: 2.2 | 0.9 | 1.98 |
Get rid of all this logic too | 0.52content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 7 wordValue: 0.1 result: 0.52 | 0.6 | 0.312 |
Max i guess config so that partners can budget their token use. | 0.83content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 12 wordValue: 0.1 result: 0.83 | 0.4 | 0.332 |
Default should be the max for the model we are using? 10k seems … | 0.94content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 14 wordValue: 0.1 result: 0.94 | 0.5 | 0.47 |
This could have been worded better. I meant that it would automa… | 5.01content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 100 wordValue: 0.1 result: 5.01 | 0.6 | 3.006 |
Alright so then it seems clear to me that we can measure the dif… | 1.38content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 22 wordValue: 0.1 result: 1.38 | 0.7 | 0.966 |
I think that's all you need to do. What is the problem? | 0.88content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 13 wordValue: 0.1 result: 0.88 | 0.3 | 0.264 |
[ 3.05625 WXDAI ]
@Keyrxng
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Review | Comment | 9 | 3.05625 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
These `pulls` rely on searching the repo for all PRs the… | 7.51content: content: p: score: 0 elementCount: 3 result: 0 regex: wordCount: 161 wordValue: 0.1 result: 7.51 | 0.9 | 1.69225 |
Is there a specific reason that you change this? The template us… | 1.7content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 28 wordValue: 0.1 result: 1.7 | 0.3 | 0.1325 |
should move this into `config` and pass this via the yml… | 0.77content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 11 wordValue: 0.1 result: 0.77 | 0.8 | 0.1515 |
Same here. Should also be renamed to `MAX_COMPLETION_TOKENS&… | 0.52content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 7 wordValue: 0.1 result: 0.52 | 0.5 | 0.065 |
This lib is only called twice and can be done just as simply wit… | 2.59content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 46 wordValue: 0.1 result: 2.59 | 0.4 | 0.2615 |
Might be best to wrap this in a try-catch because if e.g the rep… | 1.44content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 23 wordValue: 0.1 result: 1.44 | 0.7 | 0.252 |
Could be better named as it's actually fetching the linked issue… | 1.11content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 17 wordValue: 0.1 result: 1.11 | 0.6 | 0.169 |
Might be best to add handling for url matching too | 0.71content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 10 wordValue: 0.1 result: 0.71 | 0.8 | 0.1445 |
The plugin-template comes with `:4000` as standard so it… | 3.66content: content: p: score: 0 elementCount: 2 result: 0 regex: wordCount: 69 wordValue: 0.1 result: 3.66 | 0.2 | 0.188 |
[ 0.156 WXDAI ]
@gentlementlegen
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Review | Comment | 1 | 0.156 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
Does it combine this + gitignore? | 0.39content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 5 wordValue: 0.1 result: 0.39 | 0.4 | 0.156 |
Assign isn't working on pull linked i thought we fixed this @gentlementlegen |
@sshivaditya2019 the deadline is at Mon, Oct 28, 5:11 AM UTC |
|
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Task | 1 | 150 |
Review | Comment | 15 | 0 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
Resolves #19- Introduces a method to selectively retrieve diff… | 2content: content: p: score: 0 elementCount: 3 ul: score: 1 elementCount: 1 li: score: 0.5 elementCount: 2 result: 2 regex: wordCount: 22 wordValue: 0 result: 0 | 0.8 | 0 |
It uses the gitignore from the Repository if one is present. | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 11 wordValue: 0 result: 0 | 0.5 | 0 |
Revised the approach:- Sorts the diffs in ascending order base… | 2content: content: p: score: 0 elementCount: 4 ul: score: 1 elementCount: 1 li: score: 0.5 elementCount: 2 result: 2 regex: wordCount: 38 wordValue: 0 result: 0 | 0.9 | 0 |
Switched to GraphQL to directly fetch pull requests that close i… | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 37 wordValue: 0 result: 0 | 0.7 | 0 |
Is port 4000 required by some other service/kernel? I'm currentl… | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 22 wordValue: 0 result: 0 | 0.4 | 0 |
Not applicable - replaced with GraphQL solution | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 6 wordValue: 0 result: 0 | 0.6 | 0 |
It used to, I have removed all of that now. It just checks for f… | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 19 wordValue: 0 result: 0 | 0.5 | 0 |
10k seemed like a reasonable limit. If someone is using older Op… | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 26 wordValue: 0 result: 0 | 0.3 | 0 |
QA: [issue](https://github.com/sshivaditya2019/test-public/issue… | 5content: content: p: score: 0 elementCount: 2 a: score: 5 elementCount: 1 result: 5 regex: wordCount: 13 wordValue: 0 result: 0 | 0.1 | 0 |
QA: [Issue](https://github.com/sshivaditya2019/test-public/issue… | 5content: content: p: score: 0 elementCount: 1 a: score: 5 elementCount: 1 result: 5 regex: wordCount: 2 wordValue: 0 result: 0 | 0.1 | 0 |
I think that counting the number of changes isn't the best metri… | 5content: content: p: score: 0 elementCount: 2 a: score: 5 elementCount: 1 result: 5 regex: wordCount: 47 wordValue: 0 result: 0 | 0.7 | 0 |
It seems to use file size as a criterion to determine whether to… | 0content: content: p: score: 0 elementCount: 2 result: 0 regex: wordCount: 89 wordValue: 0 result: 0 | 0.6 | 0 |
@0x4007 I need some clarification. Currently, the system retriev… | 0content: content: p: score: 0 elementCount: 2 result: 0 regex: wordCount: 97 wordValue: 0 result: 0 | 0.8 | 0 |
Ok, I see now that I misunderstood these lines. I think this PR … | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 25 wordValue: 0 result: 0 | 0.5 | 0 |
QA: [Link](https://github.com/sshivaditya2019/test-public/issues… | 5content: content: p: score: 0 elementCount: 1 a: score: 5 elementCount: 1 result: 5 regex: wordCount: 2 wordValue: 0 result: 0 | 0.1 | 0 |
[ 25.327 WXDAI ]
@0x4007
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Specification | 1 | 9.18 |
Issue | Comment | 1 | 0.264 |
Review | Comment | 10 | 15.883 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
This is an example of a diff q&a gone bad. Too much was sent… | 3.06content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 56 wordValue: 0.1 result: 3.06 | 1 | 9.18 |
Assign isn't working on pull linked i thought we fixed this @gen… | 0.88content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 13 wordValue: 0.1 result: 0.88 | 0.3 | 0.264 |
Perhaps my spec isnt clear but youre not following it. - Remove… | 5.97content: content: p: score: 0 elementCount: 1 ul: score: 1 elementCount: 1 li: score: 0.5 elementCount: 4 result: 3 regex: wordCount: 54 wordValue: 0.1 result: 2.97 | 1 | 5.97 |
Should definitely be dynamic from git ignore and such. Otherwise… | 1.11content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 17 wordValue: 0.1 result: 1.11 | 0.8 | 0.888 |
Wrong approach. Should not be any hard cut off from some arbitra… | 1.06content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 16 wordValue: 0.1 result: 1.06 | 0.6 | 0.636 |
Hey i just realized this is pointless. If its ignored it wont b… | 2.2content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 38 wordValue: 0.1 result: 2.2 | 0.9 | 1.98 |
Get rid of all this logic too | 0.52content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 7 wordValue: 0.1 result: 0.52 | 0.7 | 0.364 |
Max i guess config so that partners can budget their token use. | 0.83content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 12 wordValue: 0.1 result: 0.83 | 0.3 | 0.249 |
Default should be the max for the model we are using? 10k seems … | 0.94content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 14 wordValue: 0.1 result: 0.94 | 0.5 | 0.47 |
This could have been worded better. I meant that it would automa… | 5.01content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 100 wordValue: 0.1 result: 5.01 | 0.8 | 4.008 |
Alright so then it seems clear to me that we can measure the dif… | 1.38content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 22 wordValue: 0.1 result: 1.38 | 0.7 | 0.966 |
I think that's all you need to do. What is the problem? | 0.88content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 13 wordValue: 0.1 result: 0.88 | 0.4 | 0.352 |
[ 3.02275 WXDAI ]
@Keyrxng
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Review | Comment | 9 | 3.02275 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
These `pulls` rely on searching the repo for all PRs the… | 7.51content: content: p: score: 0 elementCount: 3 result: 0 regex: wordCount: 161 wordValue: 0.1 result: 7.51 | 0.9 | 1.69225 |
Is there a specific reason that you change this? The template us… | 1.7content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 28 wordValue: 0.1 result: 1.7 | 0.3 | 0.1325 |
should move this into `config` and pass this via the yml… | 0.77content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 11 wordValue: 0.1 result: 0.77 | 0.7 | 0.13225 |
Same here. Should also be renamed to `MAX_COMPLETION_TOKENS&… | 0.52content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 7 wordValue: 0.1 result: 0.52 | 0.6 | 0.078 |
This lib is only called twice and can be done just as simply wit… | 2.59content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 46 wordValue: 0.1 result: 2.59 | 0.4 | 0.2615 |
Might be best to wrap this in a try-catch because if e.g the rep… | 1.44content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 23 wordValue: 0.1 result: 1.44 | 0.8 | 0.288 |
Could be better named as it's actually fetching the linked issue… | 1.11content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 17 wordValue: 0.1 result: 1.11 | 0.5 | 0.14125 |
Might be best to add handling for url matching too | 0.71content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 10 wordValue: 0.1 result: 0.71 | 0.6 | 0.109 |
The plugin-template comes with `:4000` as standard so it… | 3.66content: content: p: score: 0 elementCount: 2 result: 0 regex: wordCount: 69 wordValue: 0.1 result: 3.66 | 0.2 | 0.188 |
[ 0.156 WXDAI ]
@gentlementlegen
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Review | Comment | 1 | 0.156 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
Does it combine this + gitignore? | 0.39content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 5 wordValue: 0.1 result: 0.39 | 0.4 | 0.156 |
@0x4007 Seems to work: ubiquity-os-marketplace/text-conversation-rewards#77 (comment) ; there might be another reason for this. |
Could be the user configured privacy settings we discussed in another thread. We should throw a warning when we see the negative graphql response due to privacy settings. |
I will check the logs for details, hopefully more information is given there so we can determine the cause programmatically and take action. |
No POST was received meaning the event was never forwarded So it seems the Error message:
|
This is an example of a diff q&a gone bad. Too much was sent to the model and it was rate limited. Below describes a dynamic strategy that should be simple to implement and should intelligently include only the most relevant changes.
ubiquity-os-marketplace/text-conversation-rewards#163 (comment)
The text was updated successfully, but these errors were encountered: