-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
🎉 (grapher) round to significant figures / TAS-537 #3709
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @sophiamersmann and the rest of your teammates on Graphite |
Quick links (staging server):
Login:
SVG tester:Number of differences (default views): 0 ✅ Edited: 2024-06-14 08:56:02 UTC |
b2f53b7
to
a3502f6
Compare
a3502f6
to
4a01890
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice work! I had a couple of minor comments but nothing major. I didn't look in too much detail at the rendering code for the different charts but played a little with it and it all looks good.
4a01890
to
9e7ec42
Compare
Thank you for the review! Being able to type |
Adds support for rounding to significant numbers, resolves #1851
Summary
variable.display.roundingMode
andvariable.display.numSignificantFigures
roundingMode
is one ofdecimalPlaces
orsignificantFigures
(withdecimalPlaces
being the default)roundingMode
isdecimalPlaces
, we look atnumDecimalPlaces
; if theroundingMode
issignificantFigures
, we look atnumSignificantFigures
r
mode ("decimal notation, rounded to significant digits"); when rounding to sig figs, trailing zeroes are always shownDetails
scatterPointLabelStrategy
is set tox
ory
)numDecimalPlaces
is respected here)Caveats
Testing
To do
variable.display.roundingMode
andvariable.display.numSignificantFigures