Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Add test_hlines to test_datetimes.py#27211
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
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.
I think we also wish to test thexmin
/xmax
as dates here
@ksunden@scottshambaugh could I please have some advice on why the codecov/project/tests check is not passing? It's at 98.76% "compared toa589856" - I don't know why it's comparing this to the test for scatter. |
If anybody is having the same issue, I think it stemmed simply from the fact that my branch was not current with the main body of work. Synchronizing it fixed the issue. |
PR summary
I have added a datetime smoketest for
Axes.hlines
tolib/matplotlib/tests/test_datetime.py
. This example is very similar toAxes.axhline
(#27177) – but to test the entire functionality ofhlines
, I have done a combination of datetime.datetime/np.datetime64 arrays with both scalar and arrayxmin
/xmax
arguments.Below is an image of the plot generated from this example code.

This addresses the
Axes.hlines
task from#26864.PR checklist