Skip to content
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

Incorrect values (# bounties completed and $ rewarded) in the Leaderboards #1506

Closed
bockp opened this issue Jun 19, 2018 · 7 comments
Closed
Assignees
Labels
bounties bug This is something that isn't working as intended.

Comments

@bockp
Copy link

bockp commented Jun 19, 2018

Description

Type: Bug

After completing 0 bounties ( 2 started, 2 submitted, 0 "accepted"), my Leaderboard position is listed as having completed 3, and the amount listed as earned is in conflict with the Collaborative contest having been cancelled.

Technically, I participated and completed 2 bounties, but one of them was listed as traditional ( then changed to Competition, which I didn't win) and the second remains listed as Traditional, though it was a collaborative bounty that was canceled (and the reward distributed as Tips to the participants) due to the absence of bulk payouts #1422

There are two bugs associated with this, and after some study it appears both are linked, so I'll write a general description in this issue:

Bug 1 : Number of completed bounties on Leaderboard is incorrect.
Bug 2 : Amount rewarded listed in leader-board is incorrect, resulting in a higher position.

Current Behavior

On my profile page, I am currently listed as having completed no bounties, and thus having received no reward.

payout for my only completed bounty was done through a Google Form due to a lack of multi/bulk payouts at the time (Ref: #1422) and the bounty was thus canceled.

I'd therefore assume that, as correctly listed on my profile page:

image

My placement on the leaderboard would be very low.

Much to my surprise, when I checked the leaderboard, I was #12 !

image

This looks like it's taken into account not only both bounties (the contest bounty I didn't win and the cancelled collaborative one) as completed, but also counted my Tip, and some more !

I made that calculation by looking at the $ value the Leaderboard believes I've amassed, and the number of bounties it believes I've completed:

  • The Leaderboard lists 3 bounties, which I believe corresponds to how the Competition was at first listed as a Traditional project, which I Started work on, then I Stopped work once we realized the category was wrong. I then Started (and Submitted) work on the project after it was overwritten to count as a Competition project. these 2 submission + the work submitted on the second bounty == 3 total bounties

  • The amounts themselves were as follows ->

0.04 Eth * 2 (two submission to the one Competition ) + 1.1 Eth (total bounty for the cancelled Collaborative bounty) + 0.1 Eth (What I actually was Tipped for payment for my share of the Collaborative bounty) = 1.28 Ether ~= 682$

Which is approx. equal to the amount listed in the Leaderboard, ~697$

Expected Behavior

The expected behavior, of course, would have been that my Leaderboard position listed me as having completed 0 bounties (at most 1 really, since the second was cancelled), earned ~58$ (if tips are counted, 0$ otherwise and thus my actual position on the leaderboard should be at the bottom somewhere.

Reproduction

For reproduction, here's a tentative guide (I can't really test it without potentially annoying some bounty submitters by starting-stopping work ):

  1. Check leaderboard status
  2. Start work on a project marked as traditional
  3. Stop work on said project
  4. Start work again
  5. Submit work.
  6. Check leaderboard status

This would check if the bug happens no matter if there was a project category change or not. AKA., does it count in the leaderboard as 2 completed bounties (it is probably necessary to wait till the bounty is done / mark the bounty as done to see the results I'd assume)

  1. check leaderboad $ amount
  2. Start work on a project
  3. Submit work on a project
  4. Get owner of project to Cancel / award money to someone else.
  5. Check leaderboard $ amount

It's probably a good idea to check if it's just when a project is cancelled with submitted work that it counts, or if it's also counted even though you weren't the one paid out to.

Solution

Summary:

I'd suggest going through the Leaderboard code, seeing how it counts completed bounties, $ amount earned... and checking to see if the logic used takes into account cancelled bounties, collaborative/competition type projects, etc... or just awards everything to everyone who so much as touched the bounty (AKA., Started work on it).

Definition of Done

When I am again classified as a 0.1 Eth earning Leaderboard Minnow ;)

More seriously, when the $ amount and number of bounties completed properly reflect the profile page and the actual work done, and is capable of taking into account Competition/Collaborative bounties that might split the Bounty Amount between participants.

Additional Information

https://gitcoin.co/issue/gitcoinco/skunkworks/75/552
Collaborative project with the 1.1 Eth reward, of which only I Submitted the work before it was cancelled.
image

https://gitcoin.co/issue/gitcoinco/gitcoinco/42/563
GIF contest, which I didn't win but still submitted work for once, but Started-Stopped in the middle
image

This one also has Fenxxx encountering an error after sending several submission transactions, which I believe is already documented somewhere on the Issues page.

https://gitcoin.co/profile/bockp
my profile

https://gitcoin.co/leaderboard
The Leaderboard

@owocki
Copy link
Contributor

owocki commented Jun 19, 2018

good detective work! you are correct that this bug has it's roots in the recently launched contest bounties and how it was paid out.

im going to try and find some cycles later to put in a root cause fix

@PixelantDesign
Copy link
Contributor

Thanks for the extremely thorough bug report @bockp!

@mbeacom mbeacom added bug This is something that isn't working as intended. bounties leaderboard labels Jun 25, 2018
@owocki
Copy link
Contributor

owocki commented Jun 26, 2018

didn't forget about this one; just been AFK for a few days.. its on the TODO list

@PixelantDesign PixelantDesign added this to the July 2018 milestone Jul 5, 2018
@mbeacom
Copy link
Contributor

mbeacom commented Jul 5, 2018

@bockp Thanks for the detailed bug report! I just wanted to ping you here and let you know we haven't forgotten about this issue. We'll try to have a fix out shortly!

Thanks again!

@bockp
Copy link
Author

bockp commented Jul 5, 2018

@mbeacom No worries, I trust you're doing all you can :)
From what I've seen, you've got an excellent team 👍

@owocki owocki closed this as completed in e6c75aa Jul 6, 2018
@ghost ghost removed the to do label Jul 6, 2018
@owocki
Copy link
Contributor

owocki commented Jul 6, 2018

e6c75aa should fix this

@owocki
Copy link
Contributor

owocki commented Jul 6, 2018

a7baae3 too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounties bug This is something that isn't working as intended.
Projects
None yet
Development

No branches or pull requests

5 participants