Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Add default scatter marker option to rcParams #6328
Conversation
mdboom
added the
needs_review
label
Apr 24, 2016
|
The AppVeyor build looks like it failed because it took too long to download freetype. |
tacaswell
added this to the
2.0 (style change major release)
milestone
Apr 26, 2016
|
Can you also add this key to |
|
Alright, done. |
tacaswell
commented on an outdated diff
Apr 26, 2016
|
Whoops, my mistake. Fixed. |
|
One more house keeping thing, can you add a note in https://github.com/matplotlib/matplotlib/blob/master/doc/users/whats_new/rcparams.rst documenting this feature? |
|
Sure, added that to the docs. |
tacaswell
merged commit 585d128
into matplotlib:master
Apr 29, 2016
tacaswell
removed the
needs_review
label
Apr 29, 2016
tacaswell
added a commit
that referenced
this pull request
Apr 29, 2016
|
|
tacaswell |
ad81e3f
|
|
Thanks! backported to v2.x as ad81e3f I think this is your first contribution to mpl. Congratulations, hopefully we will see you again |
efiring
referenced
this pull request
May 2, 2016
Closed
Ability to set default scatter marker in matplotlibrc #6272
QuLogic
added the
Rcparams
label
May 2, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
bcongdon commentedApr 24, 2016
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.