Skip to content

[charts] Pass the axis index to extremum getter#14641

Merged
alexfauquette merged 1 commit intomui:masterfrom
alexfauquette:pass-index
Sep 16, 2024
Merged

[charts] Pass the axis index to extremum getter#14641
alexfauquette merged 1 commit intomui:masterfrom
alexfauquette:pass-index

Conversation

@alexfauquette
Copy link
Copy Markdown
Member

Part of the Radar chart.

The radar will have one radial axis per radar direction, such that use can plot on different direction values with different scales. For example comparing the GDP/capit and the life expectancy of countries on the same radar implies different scales.

To map series value to the right axis, I need their index.

So for consistency I'm providing the same values to the cartesian axes

@alexfauquette alexfauquette added the scope: charts Changes related to the charts. label Sep 16, 2024
@mui-bot
Copy link
Copy Markdown

mui-bot commented Sep 16, 2024

Deploy preview: https://deploy-preview-14641--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against ebaece9

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Sep 16, 2024

CodSpeed Performance Report

Merging #14641 will not alter performance

Comparing alexfauquette:pass-index (ebaece9) with master (c54f01d)

Summary

✅ 3 untouched benchmarks

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

Labels

scope: charts Changes related to the charts.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants