-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Y Axes alignment option: addendum #1033
Conversation
separated the code for aligning axes into function alignYAxisRanges to reduce cyclomatic complexity
Just as a progress report, I'm merging this in the hotfix/2.0.0-beta.32 branch. Here is a matrix of the 25 cases which I'm adding as an example and visual test case. (There are at least 25 cases because left and right can each be all positive, mostly positive, even positive/negative, mostly negative, or all negative.) Looks like the current heuristic gets many cases well, but severely squishes one series or the other in about 8 cases, and loses one or the other in two cases. I'm going to try a more comprehensive approach. |
Improved algorithm: |
Live in 2.0 beta 32. Thanks @gazal-k! |
Follow up to #985