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

For scaleType Interval - start y-Axis at lowest value (instead of 0) #1500

Closed
sosiology opened this issue May 6, 2024 · 2 comments · Fixed by #1565
Closed

For scaleType Interval - start y-Axis at lowest value (instead of 0) #1500

sosiology opened this issue May 6, 2024 · 2 comments · Fixed by #1565
Assignees

Comments

@sosiology
Copy link
Contributor

DoD
The y-Axis of chart dimensions of scaleType Interval starts at the lowest value (according to the available data). Dimensions of Ratio scale keep their current behaviour and start at 0.

Is your feature request related to a problem? Please describe.
If there is a visualization, where the relevant y-axis is of scaleType Interval values are typically in a limited range of values that is unrelated to zero, letting the y-axis start at zero is not helpful for the visualization. See screenshot:

Image

Describe the solution you'd like
For IntervalScale Dimensions there could be a "auto-tune" function to detect the right value range for the y-axis depending on the data that would be displayed. But this function should be only for IntervalScale, while for RatioScale, it is very important that the y-axis starts exactly at zero no matter what the data is. An example how it should loke with "autotune" is given below:

Image

Usecases and impact
I think there must be tons of visualizations that would benefit from such a behaviour.

updated description
this updated description is provided and coordinated with Sabine and Tobi

related but outdated_ https://github.com/visualize-admin/visualization-tool/issues/1255Usecases and impact

@sosiology
Copy link
Contributor Author

@bprusinowski can you provide a dataset to test this with?

@bprusinowski
Copy link
Collaborator

@sosiology I've added the How to test description to the #1565 (sorry that it wasn't there in the first place).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants