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 put png on canvas #167

Merged
merged 2 commits into from
Apr 4, 2017
Merged

Fix put png on canvas #167

merged 2 commits into from
Apr 4, 2017

Conversation

danlipsa
Copy link
Contributor

@danlipsa danlipsa commented Apr 4, 2017

No description provided.

danlipsa added 2 commits April 4, 2017 11:56
In a multi-plot with two rows, both having a png and a plot displyed in
the following order:
png, plot, png, plot

the second png ended up on top of the second plot because the colorbar raised the
png layer to 200+ while the plot layer stayed at 30+.
@danlipsa
Copy link
Contributor Author

danlipsa commented Apr 4, 2017

Goes with
CDAT/uvcdat-testdata#163

@danlipsa
Copy link
Contributor Author

danlipsa commented Apr 4, 2017

Fixes
#155

@danlipsa
Copy link
Contributor Author

danlipsa commented Apr 4, 2017

@aashish24 @doutriaux1 Please review, See end of #155 for context.

@durack1
Copy link
Member

durack1 commented Apr 4, 2017

@danlipsa @doutriaux1 I would love to try this out.. Is nightly a possibility?

@danlipsa
Copy link
Contributor Author

danlipsa commented Apr 4, 2017

@durack1 Only after it's merged.

@doutriaux1 doutriaux1 merged commit 7259eb9 into master Apr 4, 2017
@doutriaux1 doutriaux1 deleted the fix-put_png_on_canvas branch April 4, 2017 19:43
@durack1
Copy link
Member

durack1 commented Apr 4, 2017

@doutriaux1 great! So how do I check to see whether a new vtk nightly has been built? It seems that the existing one is 12+ days old on linux-64

@doutriaux1
Copy link
Contributor

@durack1 no need for a new vtk, I'll push a new vcs on conda right away

@durack1
Copy link
Member

durack1 commented Apr 4, 2017

@doutriaux1 great, can you also kick off a new vcs-nox too? And what is vcs-js? Should that also be updated?

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

Successfully merging this pull request may close these issues.

3 participants