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

inverted levels do not plot properly #637

Closed
doutriaux1 opened this issue Sep 11, 2014 · 7 comments
Closed

inverted levels do not plot properly #637

doutriaux1 opened this issue Sep 11, 2014 · 7 comments
Assignees
Milestone

Comments

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Sep 11, 2014

when data are read from 1000 -> 0 plots are not rendered correctly

@doutriaux1 doutriaux1 self-assigned this Sep 11, 2014
@doutriaux1 doutriaux1 added this to the 2.0.0 milestone Sep 11, 2014
@doutriaux1
Copy link
Contributor Author

@doutriaux1 doutriaux1 commented Sep 11, 2014

@aashish24 I need to flip Y axis on this image to go from 1000 to 0
levels

@doutriaux1
Copy link
Contributor Author

@doutriaux1 doutriaux1 commented Sep 11, 2014

currently this is what we get:
levels

@doutriaux1
Copy link
Contributor Author

@doutriaux1 doutriaux1 commented Sep 14, 2014

fixed in issue_637_inverted_axis
This was a serious bug that @painter1 uncovered! Revealed a lots of issues! Thx Jeff!
@aashish24 please review and let know @williams13 when he can test it!

@durack1
Copy link
Member

@durack1 durack1 commented Sep 14, 2014

Did the levels bug have anything to do with the error reported at PCMDI/pcmdi_metrics#59 and #580?

if (type(self.levels) == TupleType):
  File "/usr/local/uvcdat/2014-08-22/lib/python2.7/site-packages/vcs/VCS_validation_functions.py", line 1206, in _getlevels
    return self._levels
AttributeError: _levels

If so let me know when this fix is committed and I'll have another go testing the existing portrait plot

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Sep 16, 2014

@doutriaux1 I have concern about this branch.Lets talk today when you get a chance. In the mean time I updated the branch to current master (rebased as it has conflicts):

issue_637_inverted_axis_updated

@doutriaux1
Copy link
Contributor Author

@doutriaux1 doutriaux1 commented Sep 16, 2014

merged in master

@doutriaux1 doutriaux1 closed this Sep 16, 2014
@durack1
Copy link
Member

@durack1 durack1 commented Sep 16, 2014

@doutriaux1 did this have anything to do with the VCS problem decribed in PCMDI/pcmdi_metrics#59 and #580?

Just wondering whether I should rebuild and try again to test things out..

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

Successfully merging a pull request may close this issue.

None yet
3 participants