8000 Raise warning if both c and facecolors are used in scatter plot (... and related improvements in the test suite). by NGWi · Pull Request #29130 · matplotlib/matplotlib · GitHub

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

Merged

story645 merged 21 commits intomatplotlib:mainfrom

NGWi:issue24404_addwarning

Jan 3, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

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

Commits on Nov 12, 2024

Raise warning if both c and facecolors are used in scatter plot. In the future we may implement c as edgecolors in that case or cause plot to fail.

Corrected flake8 formatting warnings.

Commits on Nov 13, 2024

Removed offending whitespace and extra line.

Merge branch 'issue24404_addwarning' of https://github.com/NGWi/matplotlib into issue24404_addwarning

Corrected old test that qualified for the new warning.

Commits on Nov 18, 2024

Made message more concise, and straightened old test, to fulfill @timhoffm's points.

Added test.

Corrected flake8 linting objections.

Added direct test of _parse method to attempt to make Codecov happy.

Commits on Dec 3, 2024

Added pragma tags for code coverage to ignore unused but untested simplified return.

Commits on Dec 4, 2024