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

Adjust rescale values for small/large numbers #1226

Open
2 tasks
aboydnw opened this issue Oct 29, 2024 · 2 comments · May be fixed by #1272
Open
2 tasks

Adjust rescale values for small/large numbers #1226

aboydnw opened this issue Oct 29, 2024 · 2 comments · May be fixed by #1272
Assignees

Comments

@aboydnw
Copy link
Contributor

aboydnw commented Oct 29, 2024

Known Issues:

  1. The UI isn't great if you are trying to edit rescale values for long numbers (small decimals or large numbers)
  2. When the rescale values are very small decimals (<0.01) then the UI doesn't function properly

image

Acceptance Criteria

  • Rescale value range slider still functions for small decimals of <0.01
  • Large or long decimals are easily viewed or edited in the input box
@snmln
Copy link
Contributor

snmln commented Oct 30, 2024

Another issue Identified while working through the bug identified here: #1224 (comment)

When the values start to travers large decimal values the representation of the data changes from numeric to exponential:

Screenshot 2024-10-30 at 10 56 10 AM

We should nail down if we want this to be how we present the data value to end users.

@aboydnw
Copy link
Contributor Author

aboydnw commented Nov 12, 2024

Another thing that Sid caught that might be good to update if it's easy:

When we change the rescale values (e.g. 50 to 43) should it also get reflected at the bottom? Every time it shows 50 what we have defined in the mdx.

Image

@snmln snmln linked a pull request Nov 22, 2024 that will close this issue
@snmln snmln linked a pull request Nov 27, 2024 that will close this issue
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 a pull request may close this issue.

2 participants