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
fix: node pie graphs to use consistent powers in divisor #4726
Conversation
Welcome @laurelnaiad! |
1. The code determining whether or not the units of requests, limits and capacity required normalization was leaving out capacity in its determination. As such, if the capacity was of different units than requests and limits, they'd be left inconsistent and the percentage would be off by at least a factor of 1000 or 1024. 2. The code normalizing units was erroneously reversing a value in one conditional, and thus erroneously triggering a second conditional which was reversing the work of the first one (multiplying by the same factor by which it had just divided). Rather than fix the bug, it's easier and better to just eliminate this code. The conditionals aren't even necessary, because negative exponentiation "just works".
This is a fix for #4725 . I marked it as "Closes #...", but the robot didn't like that. :) |
Codecov Report
@@ Coverage Diff @@
## master #4726 +/- ##
==========================================
+ Coverage 45.39% 45.44% +0.04%
==========================================
Files 214 214
Lines 9976 9971 -5
Branches 96 94 -2
==========================================
+ Hits 4529 4531 +2
+ Misses 5181 5176 -5
+ Partials 266 264 -2
Continue to review full report at Codecov.
|
Closes #4725 Thanks! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: floreks, laurelnaiad The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
capacity required normalization was leaving out capacity in its
determination. As such, if the capacity was of different units than
requests and limits, they'd be left inconsistent and the percentage
would be off by at least a factor of 1000 or 1024.
conditional, and thus erroneously triggering a second conditional which
was reversing the work of the first one (multiplying by the same factor
by which it had just divided). Rather than fix the bug, it's easier and
better to just eliminate this code. The conditionals aren't even necessary,
because negative exponentiation "just works".