-
Notifications
You must be signed in to change notification settings - Fork 29
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
Crediting for unique links only #155
Comments
/start |
Tip
|
@0x4007 Seeing the spec being modified by the bot, it actually adds two links to your credit. Should this be counted? I am not sure how we can handle that. |
Yeah thats tough. I suppose the direct solution would be a config to disable footnote link credits (find lines prefixed with |
+ Evaluating results. Please wait... |
|
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Task | 1 | 75 |
Issue | Comment | 1 | 1.35 |
Review | Comment | 8 | 0 |
Conversation Incentives
Comment | Formatting | Relevance | Priority | Reward |
---|---|---|---|---|
@0x4007 Seeing the spec being modified by the bot, it actually a… | 1.8content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 30 wordValue: 0.1 result: 1.8 | 1 | 3 | 1.35 |
Resolves #155 Depends on https://github.com/ubiquity-os-market… | 15content: content: p: score: 0 elementCount: 5 a: score: 5 elementCount: 3 result: 15 regex: wordCount: 11 wordValue: 0 result: 0 | 0.2 | 3 | 0 |
The problem with that is the bot comments multiple times and you… | 5content: content: p: score: 0 elementCount: 4 a: score: 5 elementCount: 1 result: 5 regex: wordCount: 75 wordValue: 0 result: 0 | 0.6 | 3 | 0 |
I checked other comments and some are mixed, some are not. Maybe… | 5content: content: p: score: 0 elementCount: 1 a: score: 5 elementCount: 1 result: 5 regex: wordCount: 56 wordValue: 0 result: 0 | 0.7 | 3 | 0 |
Reliably depends mostly on GitHub API I suppose, sometimes I mus… | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 36 wordValue: 0 result: 0 | 0.5 | 3 | 0 |
Then it makes more sense to keep it generic for now and remove t… | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 28 wordValue: 0 result: 0 | 0.9 | 3 | 0 |
@whilefoo If you can have a look | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 7 wordValue: 0 result: 0 | 0.1 | 3 | 0 |
@0x4007 👀 | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 1 wordValue: 0 result: 0 | 0.1 | 3 | 0 |
I will do it separately because that includes multiple different… | 0content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 36 wordValue: 0 result: 0 | 0.8 | 3 | 0 |
[ 110.874 WXDAI ]
@0x4007
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Specification | 1 | 88.11 |
Issue | Comment | 1 | 3.99 |
Review | Comment | 8 | 18.774 |
Conversation Incentives
Comment | Formatting | Relevance | Priority | Reward |
---|---|---|---|---|
- When aggregating rewards, collect every link into a single dat… | 9.79content: content: ul: score: 0 elementCount: 1 li: score: 0.5 elementCount: 3 p: score: 0 elementCount: 3 a: score: 5 elementCount: 1 result: 6.5 regex: wordCount: 61 wordValue: 0.1 result: 3.29 | 1 | 3 | 88.11 |
Yeah thats tough. I suppose the direct solution would be a confi… | 1.33content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 21 wordValue: 0.1 result: 1.33 | 1 | 3 | 3.99 |
```suggestionthrow this.context.logger.error… | 0content: content: {} result: 0 regex: wordCount: 0 wordValue: 0.1 result: 0 | 0.7 | 3 | 0 |
We are intending to link an emergency triage telegram channel to… | 1.59content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 26 wordValue: 0.1 result: 1.59 | 0.6 | 3 | 2.862 |
What is elaborate in this context | 0.46content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 6 wordValue: 0.1 result: 0.46 | 0.4 | 3 | 0.552 |
This approach seems wrong. You should check revision history and… | 1.22content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 19 wordValue: 0.1 result: 1.22 | 0.8 | 3 | 2.928 |
I'm assuming this is for unique anchors. You should consider the… | 1.8content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 30 wordValue: 0.1 result: 1.8 | 0.9 | 3 | 4.86 |
@sshivaditya any ideas on how to improve the bot edit implementa… | 2.87content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 52 wordValue: 0.1 result: 2.87 | 0.5 | 3 | 4.305 |
I would try implementing the diff approach and if it is too diff… | 1.65content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 27 wordValue: 0.1 result: 1.65 | 0.5 | 3 | 2.475 |
Handle diffs in separate task if you can't do it quickly here. | 0.88content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 13 wordValue: 0.1 result: 0.88 | 0.3 | 3 | 0.792 |
[ 0.3 WXDAI ]
@Keyrxng
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Review | Comment | 1 | 0.3 |
Conversation Incentives
Comment | Formatting | Relevance | Priority | Reward |
---|---|---|---|---|
I'm happy to get back to performing reviews - I just need the gr… | 1content: content: p: score: 0 elementCount: 1 result: 0 regex: wordCount: 15 wordValue: 0.1 result: 1 | 0.1 | 3 | 0.3 |
It seems that the comment posting from the SDK does not escape the characters properly in this case. |
Similar 1
Footnotes
Missing defaults for html tags 76% ↩
The text was updated successfully, but these errors were encountered: