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

Fix a bug where x_att and y_att could end up being out of sync in image viewer #2141

Merged
merged 1 commit into from
May 15, 2020

Conversation

astrofrog
Copy link
Member

This caused issues in glue-jupyter because of the way the callbacks are set up there for the image viewer (the image data is retrieved sooner after a change in reference data)

@codecov
Copy link

codecov bot commented May 15, 2020

Codecov Report

Merging #2141 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2141   +/-   ##
=======================================
  Coverage   88.01%   88.01%           
=======================================
  Files         245      245           
  Lines       22338    22340    +2     
=======================================
+ Hits        19661    19663    +2     
  Misses       2677     2677           
Impacted Files Coverage Δ
glue/viewers/image/state.py 91.07% <100.00%> (+0.05%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 26af184...a73ab4c. Read the comment docs.

@astrofrog astrofrog merged commit 43dfea5 into glue-viz:master May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant