Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Added smoke test for Axes.imshow to test_datetime.py#27476

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

Merged

Conversation

ConstableCatnip
Copy link
Contributor

PR summary

This PR adds a smoke test forAxes.imshow in lib/matplotlib/tests/test_datetime.py and addresses taskAxes.imshow in issue#26864. Due to the behavior ofAxes.imshow with thempl.rcParams["date.converter"] = "concise", I included year, month, day, and time axis scale plots.
Below is the image of the generated plot:

image

PR checklist

Copy link
Member

@QuLogicQuLogic left a comment

Choose a reason for hiding this comment

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

I'm not too sure we need all 4 scales, as that is testing the concise date formatter, not anything specific toimshow.

… days, the axes labels now look like dates instead of just the year.
@ConstableCatnip
Copy link
ContributorAuthor

ConstableCatnip commentedDec 11, 2023
edited
Loading

I futzed with it a bit more and found that if I removed "concise" and used a tighter date range than the axis labels looked like dates (see below). Originally at first the labels were only the year number which I thought might be confusing as it didn't look like a date. Let me know what you think!

image

@ksundenksunden mentioned this pull requestDec 11, 2023
64 tasks
@QuLogicQuLogic merged commitfa6be74 intomatplotlib:mainDec 12, 2023
@QuLogic
Copy link
Member

Thanks@ConstableCatnip! Congratulations on your first PR to Matplotlib 🎉 We hope to hear from you again.

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

@QuLogicQuLogicQuLogic approved these changes

@jklymakjklymakjklymak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@ConstableCatnip@QuLogic@jklymak

[8]ページ先頭

©2009-2025 Movatter.jp