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

Properly deal with reassignment of Data.coords #1196

Merged
merged 4 commits into from
May 7, 2018

Conversation

astrofrog
Copy link
Member

@astrofrog astrofrog commented Dec 14, 2016

Re-compute world components if Data.coords is changed.

This will need to be rebased after #1195 is merged and also will have some failures we need to investigate after #1189 is merged.

@astrofrog astrofrog force-pushed the coords-reassignment branch 2 times, most recently from 2d34c11 to a0f352f Compare December 14, 2016 13:08
@astrofrog astrofrog force-pushed the coords-reassignment branch from a0f352f to 90f2d6c Compare May 7, 2018 14:30
@astrofrog
Copy link
Member Author

Still need to add a test to make sure all links based on the old world coordinates are removed

@astrofrog astrofrog force-pushed the coords-reassignment branch from c8ccaac to 0b46f7b Compare May 7, 2018 18:44
@astrofrog astrofrog merged commit 1aa3ac5 into glue-viz:master May 7, 2018
@astrofrog astrofrog added this to the v0.13.3 milestone May 8, 2018
astrofrog added a commit that referenced this pull request May 8, 2018
Properly deal with reassignment of Data.coords
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant