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 isofill levels #10

Merged
merged 4 commits into from
Sep 30, 2016
Merged

Fix isofill levels #10

merged 4 commits into from
Sep 30, 2016

Conversation

danlipsa
Copy link
Contributor

No description provided.

@danlipsa
Copy link
Contributor Author

@doutriaux1 @aashish24 Please review. Where do I commit the new png and nc for the new testfile as well as changes to uvcdat/CMakeLists.txt?

@doutriaux1
Copy link
Contributor

thanks @danlipsa please commit to the usual repo for baselines UV-CDAT/uvcdat-testdata and createa branch with this branch's name on the UVCDAT/uvcdat repo, that should trigger the test suite.

@danlipsa
Copy link
Contributor Author

@doutriaux1 Thanks.
Here are the additional PR:
CDAT/uvcdat-testdata#155
CDAT/cdat#2124

@danlipsa
Copy link
Contributor Author

@doutriaux1 @aashish24 The fix is the second commit. The first commit is identical to the one reversed.

@aashish24
Copy link
Contributor

@danlipsa in theory the code looks good to me. Basically you are setting some valid value to hidden points so that scalar range does not change. @doutriaux1 can you confirm if this fixes the issue for you?

@danlipsa
Copy link
Contributor Author

@aashish24 That is right. I also remove the globalid attribute and then add it back because otherwise RemoveDeletedCells does not remove the globalid associated with the deleted cells.

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