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

Spider Plot Index Memory #27

Open
morganjwilliams opened this issue Nov 10, 2019 · 0 comments
Open

Spider Plot Index Memory #27

morganjwilliams opened this issue Nov 10, 2019 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@morganjwilliams
Copy link
Owner

Is your feature request related to a problem? Please describe.
Currently, spider plots do not take into account previous spider plots which may have been added to an axis. Repeated calls to pyroplot.spider or pyrolite.plot.spider will thus i) reset the x-axis ticks and ticklabels to the new index (erasing records of the old) and ii) ignore the index ordering of the previous plots, such that they are likely to have different ordering.

In the meantime, if you wish to plot over existing spider plots, consider reindexing dataframes such that they're consistent throughout.

Describe the solution you'd like
Keeping a memory of previous spider plots, and using this to generate a new composite index (including all items from old and new series, and preserving old ordering for duplicated items).

Describe alternatives you've considered
This could be achieved through checking the x-axis tick labels or adding attributes to the axis directly (or through subclassing the axes, but this is unnecessary at this stage).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant