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

SWR matplotlib__matplotlib-2726 by richeshgupta · Pull Request #152 · razorpaytest/matplotlib · GitHub

Repository navigation

SWR matplotlib__matplotlib-2726 - #152

Open
richeshgupta wants to merge 1 commit into
swr-base-e586c09from
swr-head-e586c09
Open

richeshgupta wants to merge 1 commit into
swr-base-e586c09from
swr-head-e586c09

Conversation

Copy link
Copy Markdown

SWRBench case matplotlib__matplotlib-2726: pinned base 45f5c45 to head e586c09 of matplotlib#2726. Benchmark replay, not for merge.

razorpay-slash 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

Automated code review

Reviewed up to e586c09.
All 5 reviewers completed.

Comment thread lib/matplotlib/axes.py
kwargs['antialiaseds'] = False

collection = mcoll.PolyCollection(verts, **kwargs)
collection = mcoll.PolyCollection(verts, snap=False, **kwargs)

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

[MEDIUM] Calling pcolor with a snap keyword argument now raises TypeError

Issue

The explicit snap=False before **kwargs collides with any caller-supplied snap: ax.pcolor(X, Y, C, snap=True) previously forwarded snap to PolyCollection and now raises TypeError: PolyCollection() got multiple values for keyword argument 'snap'.

Evidence

SUSPECTED: A test calling ax.pcolor(x, y, C, snap=True) and expecting a returned collection would settle it; the TypeError is forced by Python whenever kwargs holds 'snap', but earlier code in pcolor may pop it before this line.

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL