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

yAxis max value is calculated incorrectly for scatterplot with multiple series when using boost module #22183

Closed
HendrikThePendric opened this issue Nov 20, 2024 · 2 comments · Fixed by #22209

Comments

@HendrikThePendric
Copy link

HendrikThePendric commented Nov 20, 2024

Background

  • We found this bug after upgrading Highcharts from v10 to v11. In v10 this is working correctly, so this is a regression in v11.
  • After the v11 upgrade the yAxis has the correct range without the boost module, but when the boost module is added the yAxis range is incorrect.

Expected behaviour

For scatterplots with multiple series and many data points the max value of the yAxis should be calculated based on the series with the largest range, as in the image below (where the boost module is not enabled):

without_boost_mode

Actual behaviour

The yAxis max is calculated based on the series with the most data-points, but with a smallest range, as in the image below (with boost module enabled):

with_boost_mode

Live demo with steps to reproduce

  1. Open this jsFiddle to see the chart with the boost module enabled, and observer the yAxis only goes up to 1400: https://jsfiddle.net/vdjsep39/2/
  2. Compare to this jsFiddle where the yAxis is scaled correctly to 5000: https://jsfiddle.net/djcx10Lh/1/

Product version

Highcharts Core 11.4.8
With Highcharts boost module (also on 11.4.8)

Affected browser(s)

All

@HendrikThePendric
Copy link
Author

We've realised that as a workaround we could use a single scatterplot series and style each individual data point. But this approach has some downsides in terms of memory usage and pre-processing of the data. Furthermore, since this is a breaking change not mentioned in the changelog, we still consider this to be a bug and prefer to await a resolution before upgrading to v11.

@hubertkozik
Copy link
Member

The regression was introduced in Highcharts v11.4.0. The last working version is Highcharts v11.3.0.

Internal note
Bisected to 74cd4c8.

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.

3 participants