-
Notifications
You must be signed in to change notification settings - Fork 68
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
Issue 1133 lon labels freaks out #1172
Conversation
making a copy of the dic before further manipulating it fixes the issue
goes with CDAT/uvcdat-testdata#29 |
@chaosphere2112 @williams13 @aashish24 @dlonie ready to be merged |
import checkimage | ||
|
||
|
||
f=cdms2.open(os.path.join(sys.prefix,"sample_data","clt.nc")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use vcs.prefix instead of sys.prefix to ensure that this will work with system python builds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dlonie good idea, we should create an issue to update all tests I'm sure it's everywhere.
👍 Looks good to me, fixes the issue we were having earlier. Sadly, does not fix my issue we found this morning, which appeared to be related 😢. |
this will ensure it will work on installation built on top of system python
Yup, this totally fixes the axis issue that @williams13 noticed; I'm going to merge! |
Issue 1133 lon labels freaks out
No description provided.