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

add test_arrow #27382

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

Closed
samchan2022 wants to merge 1 commit into matplotlib:main from samchan2022:test-add-arrow
Closed

Conversation

Copy link

@samchan2022 samchan2022 commented Nov 27, 2023

Greetings! This PR adds a smoke test for Axes.arrow in lib/matplotlib/tests/test_datetime.py. This PR closes one of the items listed in the issue #26864.
Smoke test Plot:
mpl_arrow

PR summary

PR checklist

@samchan2022 samchan2022 marked this pull request as draft November 27, 2023 21:05
Copy link
Member

ksunden commented Nov 27, 2023

how does this relate to #27372 ?

Copy link
Author

how does this relate to #27372 ?
The tests from original PR #27372 has failed just by adding a passed test case. The codecov and appveyor did not make sense. This PR was created just experiment and find out the reason behind the tests. The reason behind I believe is that I commit Not sure if there is a way to update the commit(0a4efbe) to compare with for the codecov test percent for the original PR #27372.

failed test from PR #27372
@codecov
codecov/project/tests — 98.92% (-0.09%) compared to 0a4efbe
Details
@appveyor
continuous-integration/appveyor/pr — AppVeyor build failed
Details

PR Greetings / greeting (pull_request_target)

Copy link
Member

QuLogic commented Nov 30, 2023

That failure was spurious; I'm going to close this one as a duplicate of the other, as all the discussion is happening there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /