-
Notifications
You must be signed in to change notification settings - Fork 5
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
command/[start | stop] #9
command/[start | stop] #9
Comments
@gentlementlegen so this start/stop should work exactly how the v1 Bot does currently? |
@jordan-ae Correct, the goal is to reimplement the same functionalities but in a cleaner manner within separate modules. |
/start |
Tips:
|
@gentlementlegen is there a structure yet for the XP?
Or should I cover commands for admins like:
while the default for a hunter would be the same
|
/stop |
+ You have been unassigned from the task |
/start |
Tips:
|
Is a new repo going to be created for each plugin in the wishlist or are we to open PRs against this repo? RFC @0x4007 |
Each plugin should be hosted on their own repositories. I imagine we will have a dedicated organization to host "official" plugins. |
I thought that was the plan eventually, in the meantime think you could create one for |
I am not with my computer so @gentlementlegen maybe you can. Anyways you can work anywhere and then open a pull later when the repo is ready. |
Both have been mentioned to handle XP deductions although it should only be one. I think it makes sense for #7 to have this plugin as part of it's Or I can remove the notion of XP handling in this and have it be just a slash command plugin for |
Only wildcard unassign should deduct XP for inactivity. If a user gives up with /stop within the allocated timeframe it's fine. No punishment. |
PR ready for review once the repo is good to go. I assume the org will just be |
Apologies this slipped my mind. |
Sure, maybe one of the following:
Whatever the repo gets called I'll change my code references to |
Related repository: https://github.com/ubiquibot/command-start-stop |
You can add a description under the repository description field that explains it is used for assignee management. |
No the admin can unassign using the GitHub UI. Please try and keep this codebase as simple as possible. I am keen on having the minimal amount of features necessary so that the codebase is easy to maintain. I think that the rewards shouldn't be settable as you described because that is affected by factors that have little to do with the work, like if the collaborator had a good rest, or a good day, or ate a good meal just before reviews etc. This does remind me, I think that in our "database" (JSON storage in the repository) we should include metadata on the points deductions. For example, include a GitHub issue URL and then the deduction amount. I don't think we should support additions to XP. My vision is that XP should be derived from total dollars earned, minus deductions. The reason why this logic is sound to me is because if we are trying to build meritocracies, the amount of dollars distributed should directly correlate with the value the contributor has provided. The more value they provided to the organization means that they should have more influence over the organization. XP is intended to eventually be a major factor in governance. |
Is this a replacement KV for action-only plugins sort of thing or worker types should have the same sort of storage? Until AA is finalized we'll be funding wallets with the faucet so unless the permit generation flow is modified to fund their address we'll need to either:
listed in order of preference I'm a fan of this vision as I'm in good standing with this approach lmao but it is the most provable metric to go by especially if XP is always <= dollars earned.
Great advice I need to embody more myself 😂 |
My proposal ubiquity-os/plugin-template#2
How about sponsored transactions as a first starting point instead? Is this possible?
I do like that this approach can work on any network, not just gnosis chain. However I assume it's more work to set up.
It's a pretty common negotiation tactic to make an ask over food |
More info here but yes it's possible I'm sure. Right now we have a fresh EOA with funding, we could just as easily deploy a Safe right now and use that safe as the user wallet and not the EOA. Then setup the infrastructure needed for it.
On the OpenZeppelin Defender side it's a couple click process to clone a relayer onto another chain. The faucet needs refactored to handle multiple chains and multiple relayers but it needs refactored anyway to support the first iteration of AA anyway. |
+ Evaluating results. Please wait... |
|
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Task | 1 | 800 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|
[ 0.2275 WXDAI ]
@jordan-ae
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Comment | 1 | 0.2275 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
@gentlementlegen so this start/stop should work exactly how the … | 0.325content: p: count: 13 score: 1 wordValue: 0.1 formattingMultiplier: 0.25 | 0.7 | 0.2275 |
[ 0 WXDAI ]
@gentlementlegen
Contributions Overview
View | Contribution | Count | Reward |
---|
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|
[ 0 WXDAI ]
@0x4007
Contributions Overview
View | Contribution | Count | Reward |
---|
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|
[ 7.04 WXDAI ]
@rndquu
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Review | Comment | 12 | 7.04 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
@Keyrxng Getting the following error on `yarn worker`: &… | 4.2content: p: count: 25 score: 1 code: count: 17 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
Works fine Moved this https://github.com/ubiquibot/command-start… | 1.6content: p: count: 16 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
@Keyrxng `UBIQUIBOT_TOKEN` is the github app token, righ… | 3content: p: count: 29 score: 1 code: count: 1 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
@gentlementlegen Do we need to pass `CLOUDFLARE_ACCOUNT_ID&#… | 1.3content: p: count: 10 score: 1 code: count: 1 score: 1 a: count: 2 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
@gentlementlegen The convention is to simply use the latest yarn… | 1.7content: p: count: 17 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
@Keyrxng As far as I understand `getMultiplier()`, `… | 2.2content: p: count: 19 score: 1 code: count: 3 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
@Keyrxng Could you update [worker-deploy.yml](https://github.com… | 1.1content: p: count: 10 score: 1 a: count: 1 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
Works fine with yarn v4, at least in the "worker mode". As far I… | 3.7content: p: count: 35 score: 1 a: count: 1 score: 1 code: count: 1 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
There are 3 entities here: 1. Kernel 2. Core ubiquity plugins 3.… | 11.6content: p: count: 114 score: 1 code: count: 1 score: 1 a: count: 1 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
@Keyrxng The plugin config example provided in the README throws… | 3.9content: p: count: 31 score: 1 code: count: 7 score: 1 a: count: 1 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
@Keyrxng Still getting the `SyntaxError: "[object Object]" i… | 3.1content: p: count: 23 score: 1 code: count: 7 score: 1 a: count: 1 score: 1 wordValue: 0.1 formattingMultiplier: 1 | - | - |
I'm getting this warning on `/start`: ``` #… | 8.8content: p: count: 72 score: 1 code: count: 16 score: 1 wordValue: 0.1 formattingMultiplier: 1 | 0.8 | 7.04 |
|
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Comment | 6 | 51.6 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
Each plugin should be hosted on their own repositories. I imagin... | 2.1 | 0.38 | 2.1 |
I am not with my computer so @gentlementlegen maybe you can. Any... | 2.7 | 0.37 | 2.7 |
Only wildcard unassign should deduct XP for inactivity. If a use... | 2.4 | 0.49 | 2.4 |
> Apologies this slipped my mind. `start-stop` seems ... | 14.7li: count: 3 score: "3" words: 25 code: count: 6 score: "6" words: 10 | 0.41 | 14.7 |
> @gentlementlegen is there a structure yet for the XP? >... | 22.6li: count: 2 score: "2" words: 26 | 0.45 | 22.6 |
> > "database" (JSON storage in the repository) > >... | 7.1li: count: 1 score: "1" words: 13 | 0.49 | 7.1 |
[ 27 WXDAI ]
@gentlementlegen
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Specification | 1 | 14.2 |
Issue | Comment | 3 | 12.8 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
A user should be able to start or stop working on a task, by typ... | 14.2a: count: 1 score: "1" words: 3 code: count: 2 score: "2" words: 2 | 1 | 14.2 |
@jordan-ae Correct, the goal is to reimplement the same function... | 3.8 | 0.63 | 3.8 |
Apologies this slipped my mind. `start-stop` seems like ... | 7.2code: count: 2 score: "2" words: 4 | 0.44 | 7.2 |
Related repository: https://github.com/ubiquibot/command-start-s... | 1.8 | 0.43 | 1.8 |
[ 1.4 WXDAI ]
@jordan-ae
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Comment | 1 | 1.4 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
@gentlementlegen so this start/stop should work exactly how the ... | 1.4 | 0.28 | 1.4 |
[ 899.7 WXDAI ]
@Keyrxng
Contributions Overview
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Task | 1 | 800 |
Issue | Comment | 8 | 0 |
Issue | Comment | 8 | 99.7 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
@gentlementlegen is there a structure yet for the XP?
| -li: count: 5 score: "0" words: 65 code: count: 5 score: "0" words: 11 | 0.46 | - |
- https://github.com/ubq-testing/start-stop-module/issues/4 init... | -li: count: 1 score: "0" words: 12 | 0.29 | - |
I thought that was the plan eventually, in the meantime think yo... | -code: count: 1 score: "0" words: 2 | 0.43 | - |
- #7 this plugin handles the detection and the follow-up of the ... | -li: count: 2 score: "0" words: 28 code: count: 4 score: "0" words: 6 | 0.545 | - |
PR ready for review once the repo is good to go.I assume the... | -code: count: 1 score: "0" words: 4 | 0.515 | - |
Sure, maybe one of the following:
| -li: count: 8 score: "0" words: 20 | 0.365 | - |
> "database" (JSON storage in the repository)Is this a re... | -li: count: 3 score: "0" words: 57 code: count: 2 score: "0" words: 3 | 0.395 | - |
> How about sponsored transactions as a first starting point ... | -a: count: 1 score: "0" words: 1 | 0.52 | - |
@gentlementlegen is there a structure yet for the XP?
| 22.2li: count: 5 score: "5" words: 65 code: count: 5 score: "5" words: 11 | 0.46 | 22.2 |
- https://github.com/ubq-testing/start-stop-module/issues/4 init... | 4.7li: count: 1 score: "1" words: 12 | 0.29 | 4.7 |
I thought that was the plan eventually, in the meantime think yo... | 4.1code: count: 1 score: "1" words: 2 | 0.43 | 4.1 |
- #7 this plugin handles the detection and the follow-up of the ... | 22.6li: count: 2 score: "2" words: 28 code: count: 4 score: "4" words: 6 | 0.545 | 22.6 |
PR ready for review once the repo is good to go.I assume the... | 4.2code: count: 1 score: "1" words: 4 | 0.515 | 4.2 |
Sure, maybe one of the following:
| 11.8li: count: 8 score: "8" words: 20 | 0.365 | 11.8 |
> "database" (JSON storage in the repository)Is this a re... | 20li: count: 3 score: "3" words: 57 code: count: 2 score: "2" words: 3 | 0.395 | 20 |
> How about sponsored transactions as a first starting point ... | 10.1a: count: 1 score: "1" words: 1 | 0.52 | 10.1 |
A user should be able to start or stop working on a task, by typing
/start
or/stop
within a comment under an issue.Stop this one will be especially useful to gracefully quit a task without losing XP. if somebody is disqualified through poor/non performance, then they will get negative XP.
See code reference
The text was updated successfully, but these errors were encountered: