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

Raise warning if both c and facecolors are used in scatter plot (... and related improvements in the test suite).#29130

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
story645 merged 21 commits intomatplotlib:mainfromNGWi:issue24404_addwarning
Jan 3, 2025
Merged
Changes from1 commit
Commits
Show all changes
21 commits
Select commitHold shift + click to select a range
5a26bb9
Raise warning if both c and facecolors are used in scatter plot. In t…
NGWiNov 12, 2024
f1966c2
Corrected flake8 formatting warnings.
NGWiNov 12, 2024
53a0825
Removed offending whitespace and extra line.
NGWiNov 13, 2024
a777a15
Merge branch 'issue24404_addwarning' of https://github.com/NGWi/matpl…
NGWiNov 13, 2024
8031809
Corrected old test that qualified for the new warning.
NGWiNov 13, 2024
8ae213c
Made message more concise, and straightened old test, to fulfill @tim…
NGWiNov 18, 2024
49f03f7
Added test.
NGWiNov 18, 2024
8234066
Corrected flake8 linting objections.
NGWiNov 18, 2024
38c092a
Added direct test of _parse method to attempt to make Codecov happy.
NGWiNov 18, 2024
e6e49aa
Added pragma tags for code coverage to ignore unused but untested sim…
NGWiDec 3, 2024
39e1899
Raise warning if both c and facecolors are used in scatter plot. In t…
NGWiNov 12, 2024
06c8a76
Corrected flake8 formatting warnings.
NGWiNov 12, 2024
72138c6
Corrected old test that qualified for the new warning.
NGWiNov 13, 2024
319b92b
Made message more concise, and straightened old test, to fulfill @tim…
NGWiNov 18, 2024
e348dfc
Added test.
NGWiNov 18, 2024
fcf6ce2
Corrected flake8 linting objections.
NGWiNov 18, 2024
9601975
Added direct test of _parse method to attempt to make Codecov happy.
NGWiNov 18, 2024
a18364c
Added pragma tags for code coverage to ignore unused but untested sim…
NGWiDec 3, 2024
310a15f
Corrected placement of pragma tags and added 'currently unused' to Je…
NGWiDec 4, 2024
7c01a7b
Tim's suggestion for naming test's list of colors.
NGWiDec 4, 2024
fd3d9db
Corrected call to changed variable name.
NGWiDec 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
PrevPrevious commit
NextNext commit
Raise warning if both c and facecolors are used in scatter plot. In t…
…he future we may implement c as edgecolors in that case or cause plot to fail.
  • Loading branch information
@NGWi
NGWi committedDec 4, 2024
commit39e18997a2d680fb8f2e260f4e5325f166b3a0f5
12 changes: 11 additions & 1 deletionlib/matplotlib/axes/_axes.py
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -4666,10 +4666,20 @@ def _parse_scatter_color_args(c, edgecolors, kwargs, xsize,
edgecolors = kwcolor
if facecolors is None:
facecolors = kwcolor



if edgecolors is None and not mpl.rcParams['_internal.classic_mode']:
edgecolors = mpl.rcParams['scatter.edgecolors']

# Raise a warning if both `c` and `facecolor` are set (issue #24404).
if c is not None and facecolors is not None:
_api.warn_external(
"You passed both c and facecolor/facecolors for the markers. "
"Matplotlib is ignoring the facecolor "
"in favor of what you specified in c. "
Copy link
Member

Choose a reason for hiding this comment

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

Slightly more concise:

Suggested change
"You passed both c and facecolor/facecolors for the markers. "
"Matplotlib is ignoring the facecolor "
"in favor of what you specified in c. "
"Passing both c and facecolor/facecolors for the markers "
"is ambiguous. facecolor is ignored in in favor of c. "

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Thank you for reviewing the comment. Your change is more concise, and I understand the argument that matplotlib developers consider passing both as ambiguous.
However, a user intentionally passing both will likely not understand why it is ambiguous. They probably assumed incorrectly, as the issue's Original Poster did, that c defines both a face color and edge color and that just like adding edge color works, the face color can also be overwritten.

"This behavior may change in the future."
)

c_was_none = c is None
if c is None:
c = (facecolors if facecolors is not None
Expand Down
[8]ページ先頭
©2009-2025 Movatter.jp