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

add simple success rate calculation #5104

Merged
merged 2 commits into from
Sep 4, 2019
Merged

add simple success rate calculation #5104

merged 2 commits into from
Sep 4, 2019

Conversation

danlipert
Copy link
Contributor

Description

Adds success rate calculation based on already existing calculation found in quarterly stats function

Refers/Fixes

exposes success rate for #5036

Testing

Tested locally:
Screen Shot 2019-08-30 at 5 52 28 PM

@codecov
Copy link

codecov bot commented Aug 30, 2019

Codecov Report

Merging #5104 into master will decrease coverage by <.01%.
The diff coverage is 14.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5104      +/-   ##
==========================================
- Coverage   31.03%   31.02%   -0.01%     
==========================================
  Files         218      218              
  Lines       17536    17543       +7     
  Branches     2413     2414       +1     
==========================================
+ Hits         5442     5443       +1     
- Misses      11871    11875       +4     
- Partials      223      225       +2
Impacted Files Coverage Δ
app/dashboard/models.py 56.43% <14.28%> (-0.17%) ⬇️
app/dashboard/views.py 14.08% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2489fb...1a98bbd. Read the comment docs.

@codecov
Copy link

codecov bot commented Aug 30, 2019

Codecov Report

Merging #5104 into master will increase coverage by 0.18%.
The diff coverage is 18.18%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5104      +/-   ##
==========================================
+ Coverage   31.03%   31.21%   +0.18%     
==========================================
  Files         218      218              
  Lines       17536    18492     +956     
  Branches     2413     2736     +323     
==========================================
+ Hits         5442     5772     +330     
- Misses      11871    12466     +595     
- Partials      223      254      +31
Impacted Files Coverage Δ
app/dashboard/views.py 15.71% <ø> (+1.63%) ⬆️
app/dashboard/models.py 57.67% <18.18%> (+1.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2489fb...8563ce1. Read the comment docs.

num_completed_bounties = self.bounties.filter(
idx_status__in=['done'], network=network).count()
terminal_state_bounties = self.bounties.filter(
idx_status__in=Bounty.TERMINAL_STATUSES, network=network).count()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't know how to solve the cooperative bounty case, since I can be paid out but the bounty still open.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is kind of the same situation on the dashboard, should we fetch fulfillments also?

@thelostone-mc thelostone-mc merged commit 0246637 into master Sep 4, 2019
@thelostone-mc thelostone-mc deleted the success-rate branch June 27, 2020 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants