Conversation
|
Thanks for your contribution! The pull request is marked to be Document changes are required in this PR. Please also make a PR to apache/echarts-doc for document changes and update the issue id in the PR description. When the doc PR is merged, the maintainers will remove the |
|
This PR has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If you wish not to mark it as stale, please leave a comment in this PR. We are sorry for this but 2 years is a long time and the code base has been changed a lot. Thanks for your contribution anyway. |
|
Thanks for landing the aggregate/id migration design here. Two correctness items from the experimental plugin that are worth checking against this branch before it leaves draft (they are easy to miss because
Also, this PR’s I am not proposing extra API on this PR; just flagging the SUM seed/ |
Brief Information
This pull request is in the type of:
What does this PR do?
This PR migrates the aggregate and id transform from echarts-simple-transform, which is a experiment repo maintained by one of our maintainers @100pah. The API design is mainly keeped same but code are refactored to be more clear and neater.
Also in this PR the old
seriesLayoutByoption is renamed tosourceLayoutto be more clear.Details
Aggregate transform
ID transform
Misc
Related test cases or examples to use the new APIs
N.A.
Others
Merging options
Other information