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 into

matplotlib:mainfrom

NGWi:issue24404_addwarning

Jan 3, 2025

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.
NGWi
committed
5a26bb9
Corrected flake8 formatting warnings.
NGWi
committed
f1966c2

Commits on Nov 13, 2024

Removed offending whitespace and extra line.
NGWi
authored
53a0825
Merge branch 'issue24404_addwarning' of https://github.com/NGWi/matplotlib into issue24404_addwarning
NGWi
committed
a777a15
Corrected old test that qualified for the new warning.
NGWi
committed
8031809

Commits on Nov 18, 2024

Commits on Dec 3, 2024

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

Commits on Dec 4, 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.
NGWi
committed
39e1899
Corrected flake8 formatting warnings.
NGWi
committed
06c8a76
Corrected old test that qualified for the new warning.
NGWi
committed
72138c6
Made message more concise, and straightened old test, to fulfill @timhoffm's points.
NGWi
committed
319b92b
Added test.
NGWi
committed
e348dfc
Corrected flake8 linting objections.
NGWi
committed
fcf6ce2
Added direct test of _parse method to attempt to make Codecov happy.
NGWi
committed
9601975
Added pragma tags for code coverage to ignore unused but untested simplified return.
NGWi
committed
a18364c
Corrected placement of pragma tags and added 'currently unused' to Jefro's test.
NGWi
committed
310a15f
Tim's suggestion for naming test's list of colors.

NGWi
and
timhoffm
authored
7c01a7b
Corrected call to changed variable name.
NGWi
committed
fd3d9db