Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Add default scatter marker option to rcParams#6328
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
The AppVeyor build looks like it failed because it took too long to download freetype. |
Can you also add this key to |
Alright, done. |
| @@ -461,6 +461,9 @@ backend : $TEMPLATE_BACKEND | |||
| # If Numpy 1.11 or later is | |||
| # installed, may also be `auto` | |||
| ### SCATTER PLOTS | |||
| #scatter.marker # The default marker type for scatter plots. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
This needs: o
e87dc6f tof0dcc36CompareWhoops, my mistake. Fixed. |
One more house keeping thing, can you add a note inhttps://github.com/matplotlib/matplotlib/blob/master/doc/users/whats_new/rcparams.rst documenting this feature? |
Sure, added that to the docs. |
Add default scatter marker option to rcParams
Thanks! backported to v2.x asad81e3f I think this is your first contribution to mpl. Congratulations, hopefully we will see you again 😄 |
Addresses#6272.
Removes hard-coded
'o'as the default marker from within thescatterdefinition. Adds the'scatter.marker'option inrcParams, which is then loaded whenscatteris called.This is my first contribution, so I apologize in advance if I've violated any best practices.