Skip to content

Conversation

pelson
Copy link
Member

@pelson pelson commented Oct 31, 2017

Closes #612.

Would need a what's new.

@pelson
Copy link
Member Author

pelson commented Oct 31, 2017

Looks like a genuine failure. Old test: https://scitools.github.io/test-iris-imagehash/images/f15f832a5ee0492a36e8b9ed8fa4f2b629dace4921681e17a9807e7c89835ef0.png

New test:

result-iris tests test_plot test1dplotmultiargs test_cube 0

@pelson pelson force-pushed the 1d_plot_against_yz branch from a866f7f to 7ed42da Compare October 31, 2017 21:05
@ajdawson
Copy link
Member

ajdawson commented Nov 1, 2017

That is an altltude vs temperature plot, the new one is correct for the change you made.

@ajdawson
Copy link
Member

ajdawson commented Nov 1, 2017

Others are either altitude vs temperature or latitude vs temperature, both of which should be affected by this PR, so all looks good to me.

lib/iris/plot.py Outdated
# is vertical-like, put the coordinate on the y axis, and the data o
# the x.
if (isinstance(v_object, iris.cube.Cube)
and iris.util.guess_coord_axis(u_object) in ['Y', 'Z']):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is your coding standards error, the and should be on the previous line.

def _label_1d_plot(*args):
if len(args) > 1 and isinstance(args[1],
u_obj, v_obj, _, _, _ = iplt._get_plot_objects(args)
if len(args) > 1 and isinstance(u_obj,
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If statement is now not needed.

@ajdawson
Copy link
Member

Probably a small amount of effort needed to bring this up to speed.

…otting just a cube against a vertical or y coordinate.
@pelson pelson force-pushed the 1d_plot_against_yz branch from 7ed42da to b1bfb4a Compare February 14, 2018 15:38
@pelson
Copy link
Member Author

pelson commented Feb 14, 2018

I've rebased and fixed the pep8 issue. I'm unlikely to get the time to do the image hashes soon unfortunately.

@pelson
Copy link
Member Author

pelson commented Feb 1, 2019

Closing as this has gone stale. Implementation is solid if somebody want to take it on and get it over the finishing line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants