Skip to content

[ENH]: scatter should be able to take a list of edgecolors #27242

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
story645 opened this issue Nov 1, 2023 · 4 comments
Closed

[ENH]: scatter should be able to take a list of edgecolors #27242

story645 opened this issue Nov 1, 2023 · 4 comments

Comments

@story645
Copy link
Member

story645 commented Nov 1, 2023

Problem

Scatter already takes a list of colors, would be nice if it also took a list of edgecolors since those are also colors.

Proposed solution

Allow scatter to take in and set edgecolors on a permarker basis same as it does for colors

@mwaskom
Copy link

mwaskom commented Nov 1, 2023

It might not be documented but I think it actually already does?

plt.scatter([1, 2, 3], [1, 2, 3], s=500, c="w", ec=["r", "b", "g"], lw=3)

@rcomer
Copy link
Member

rcomer commented Nov 1, 2023

Scatter doc does list an edgecolors parameter.

@story645
Copy link
Member Author

story645 commented Nov 1, 2023

Yeah I think I may be missed it cause of sleep deprivation and code not working properly for probably other reasons. - This is a thing I thought it did and then was surprised it didn't 🤦‍♀️
I'm gonna close this issue and make colors plural in the docstring to up clarity a drop.

@story645 story645 closed this as completed Nov 1, 2023
@story645
Copy link
Member Author

story645 commented Nov 1, 2023

And I can't make colors plural cause it's spelled that way to reference type, so I'm better off waiting for #27200

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants