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

Incorrect time axis units for iris.quickplot.plot #615

Closed
ajdawson opened this issue Jul 16, 2013 · 4 comments
Closed

Incorrect time axis units for iris.quickplot.plot #615

ajdawson opened this issue Jul 16, 2013 · 4 comments

Comments

@ajdawson
Copy link
Member

When making a 1D plot where time is a coordinate with iris.quickplot.plot the label for the time coordinate is "Time / units", where units is a udunits time unit e.g. days since 1900-01-01 00:00:0.0. However, since the time coordinate will have been translated into date/time for the plot the units part no longer makes sense, and should not be displayed.

An example:
time_units

@pelson
Copy link
Member

pelson commented Jul 16, 2013

I've been thinking about this recently too. A part of me is tempted to hook the label into the AutoDateFormatter and update the label with something like time / hh:mm, or whichever is (dynamically) chosen by AutoDateFormatter. Equally, it would be just as valuable at this stage for it just to say time and not bother with the units. Thoughts?

@rhattersley
Copy link
Member

Equally, it would be just as valuable at this stage for it just to say time and not bother with the units. Thoughts?

If the labels are "absolute times" (e.g. "Jan 17", "8:30am", etc) then I'd ditch the units. But there will also be occasions where we have "relative times" (e.g. a forecast period of 6 hours) so the label would just be "6" and we need the units.

@esc24
Copy link
Member

esc24 commented Jul 16, 2013

I agree with @rhattersley. is_reference_time() should do the job in iris.quickplot._title().

@ajdawson
Copy link
Member Author

To be clear I was definitely only referring to time coordinates where a reference unit is used, and that are automatically transformed to absolute dates/times. Other times should preserve units.

esc24 added a commit that referenced this issue Jul 17, 2013
Remove quickplot unit label for time reference coordinates. Closes #615.
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

No branches or pull requests

4 participants