[charts] Introduce the radar chart #16406
Open
+2,524
−103
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
alexfauquette
added
new feature
|
Deploy preview: https://deploy-preview-16406--material-ui-x.netlify.app/ Updated pages: |
alexfauquette
force-pushed
the
radar-chart
branch
from
37ed9fc to
a95fddc
Compare
github-actions
bot
added
the
PR: out-of-date
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
github-actions
bot
removed
the
PR: out-of-date
alexfauquette
force-pushed
the
radar-chart
branch
from
a589b3a to
e1b7842
Compare
CodSpeed Performance ReportMerging #16406 will not alter performanceComparing Summary
|
github-actions
bot
added
the
PR: out-of-date
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
github-actions
bot
removed
the
PR: out-of-date
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is a rough first version of the Radar chart #7925. Lots of features are missing.
The idea is to validate the data structure, and it's mapping to a rotation/radius scale system. If ok, we can merge this one and add features in distinct PR such that we keep them small enough to be relevant (I don't expect someone to read 200 files with 10 different features and spot any bug or room for improvement)
Main idea
The polar plugin introduce the support of rotatio/radius axes that work similarly to the x/y axes
The radar don't expose directly those axies. It uses an intermediate
metricsto make sure the radar has only one rotation axis with scale point, and one radius axis per metric.Then we have one component per item rendered (grid, area, and marks)
Features
Preview: https://deploy-preview-16406--material-ui-x.netlify.app/x/react-charts/radar/