Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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
wants to merge 1 commit into from
Closed

Conversation

samchan2022
Copy link

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
@ksunden
Copy link
Member

ksunden commented Nov 27, 2023

how does this relate to #27372 ?

@samchan2022
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)

@QuLogic
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants