Adjusting rescale value for small/large numbers #1272
Conversation
✅ Deploy Preview for veda-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
app/scripts/components/exploration/components/datasets/colorRangeSlider/color-range-slider.scss
Show resolved
Hide resolved
snmln
force-pushed
the
fix/color-range-slider
branch
from
59fc32c
to
f7016a0
Compare
snmln
requested review from
vgeorge,
AliceR,
hanbyul-here,
dzole0311,
sandrahoang686 and
aboydnw
|
@snmln I assume you're looking at NO2 diff, not CO2 diff. Just commenting here for posterity, I think this is an acceptable visualization of these small/large numbers.
Only question is, were you able to test this out where the magnitude of the min/max were both <0.01? |
|
CO2 will also display a 1.5e-6 value I believe, so both NO2 and CO2 will work for testing purposes. And yes, I was able to confirm the 2 extremes of the data visualization will present as desired. |
snmln
force-pushed
the
fix/color-range-slider
branch
from
85f994b
to
7fdb52e
Compare
migrating changes from color-maps-range-slider #1190
snmln
force-pushed
the
fix/color-range-slider
branch
from
459eef5
to
1d4d878
Compare
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
There was a problem hiding this comment.
This work was previously approved in: #1190. Then reverted when bug was identified. I needed to pull those changes back in to correctly reflect updates.
Related Ticket: #1226
Description of Changes
Notes & Questions About Changes
When observing a value in either the max or min inputs if that value has a character count of 6 or greater the component will convert that number into scientific notation as a display value.
Validation / Testing
Run this branch on the veda-config code base, navigate to the E&A page and select the difference CO2 dataset. This is an extreme data set that should display all changes in this component.