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

Retain Element._plot_id on redim #3478

Merged
merged 1 commit into from Feb 8, 2019

Conversation

Projects
None yet
1 participant
@philippjfr
Copy link
Contributor

philippjfr commented Feb 8, 2019

As long as the data itself does not change redim should not change the plot_id and therefore break links between plots. This PR checks whether the dimension names change before restoring the previous plot_id on a redim, since the only redim change that necessitates a change to the data is a change in the dimension names.

@philippjfr philippjfr added the bug label Feb 8, 2019

@philippjfr philippjfr merged commit 02cf145 into master Feb 8, 2019

6 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls First build on redim_plot_id at 89.833%
Details
s3-reference-data-cache Test data is cached.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.