FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fixed test case(test_axes.py) failing on ppc64le by ManthanIBM · Pull Request #29409 · matplotlib/matplotlib · GitHub

  1. matplotlib
  2. matplotlib

Repository navigation

Fixed test case(test_axes.py) failing on ppc64le - #29409

Merged
timhoffm merged 1 commit into
matplotlib:mainfrom
ManthanIBM:matplotlib_ppc64le_fix
Jan 6, 2025
ConversationCommits (1)ChecksFiles changed
Merged

Fixed test case(test_axes.py) failing on ppc64le#29409
timhoffm merged 1 commit into
matplotlib:mainfrom
ManthanIBM:matplotlib_ppc64le_fix

Conversation

ManthanIBM commented Jan 6, 2025

Copy link
Copy Markdown
Contributor
  • Why is this change necessary?
    This change fixes the test case failing on ppc64le architecture.
  • What problem does it solve?
    This change will pass all the test cases on power.
  • What is the reasoning for this implementation?
    This change is implemented as required architecture was not specified in the test case.

PR checklist

github-actions Bot reviewed Jan 6, 2025
View reviewed changes

github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide

We strive to be a welcoming and open project. Please follow our Code of Conduct.

ManthanIBM force-pushed the matplotlib_ppc64le_fix branch from 7fc59f1 to ef50b92 Compare January 6, 2025 07:57
ManthanIBM force-pushed the matplotlib_ppc64le_fix branch from ef50b92 to ff1b230 Compare January 6, 2025 08:12
timhoffm merged commit 64b767f into matplotlib:main Jan 6, 2025

timhoffm commented Jan 6, 2025

Copy link
Copy Markdown
Member

Thanks @ManthanIBM, and congratulations on your first contribution to Matplotlib!

QuLogic added this to the v3.10.1 milestone Jan 7, 2025

QuLogic commented Jan 7, 2025

Copy link
Copy Markdown
Member

@meeseeksdev backport to v3.10.x

meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jan 7, 2025
Backport PR matplotlib#29409: Fixed test case(test_axes.py) failing o…
0db1cce 
…n ppc64le
meeseeksmachine mentioned this pull request Jan 7, 2025
Merged
QuLogic added a commit that referenced this pull request Jan 7, 2025
Merge pull request #29417 from meeseeksmachine/auto-backport-of-pr-29…
19e4d4d 
…409-on-v3.10.x

Backport PR #29409 on branch v3.10.x (Fixed test case(test_axes.py) failing on ppc64le)
ksunden mentioned this pull request Mar 3, 2025
Merged
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

github-actions[bot] github-actions[bot] left review comments
timhoffm timhoffm approved these changes
greglucas greglucas approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v3.10.1

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL