/matplotlib

DOC: Updated docs for rc_context #6176

Merged
merged 1 commit into from Mar 20, 2016
+9 −1

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
Documentation rcparams
Projects
None yet
Milestone
  2.1 (next point r...
6 participants
@madphysicist @jenshnielsen @tacaswell @mdboom @WeatherGod @QuLogic
@madphysicist
Contributor

madphysicist commented Mar 17, 2016

Included interactive and savefig example.

mdboom added the needs_review label Mar 17, 2016

@jenshnielsen
Owner

jenshnielsen commented Mar 19, 2016

Looks good. The pep8 test fails because you have a blank line with whitespace in in.

madphysicist closed this Mar 19, 2016

mdboom removed the needs_review label Mar 19, 2016

madphysicist reopened this Mar 19, 2016

mdboom added the needs_review label Mar 19, 2016

@tacaswell tacaswell and 1 other commented on an outdated diff Mar 19, 2016

View full outdated diff
lib/matplotlib/__init__.py
@@ -1285,7 +1285,15 @@ class rc_context(object):
plt.plot(x, a)
The 'rc' dictionary takes precedence over the settings loaded from
- 'fname'. Passing a dictionary only is also valid.
+ 'fname'. Passing a dictionary only is also valid. For example a
+ common usage is
@tacaswell

tacaswell Mar 19, 2016

Owner

might need trailing :: here to make rst/sphinx happy?

@madphysicist

madphysicist Mar 19, 2016

Contributor

Fixed that too
@madphysicist
Contributor

madphysicist commented Mar 19, 2016

Thanks. I saw the error on my phone and couldn't figure out what was causing it. Fixed now.
@madphysicist @madphysicist madphysicist DOC: Updated docs for rc_context 
Included `interactive` and `savefig` example.

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
a64686e

@WeatherGod WeatherGod added a commit that referenced this pull request Mar 20, 2016

@WeatherGod WeatherGod Merge pull request #6176 from madphysicist/patch-1 
DOC: Updated docs for rc_context
 b8d8b1b

@WeatherGod WeatherGod merged commit b8d8b1b into matplotlib:master Mar 20, 2016
2 checks passed

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

mdboom removed the needs_review label Mar 20, 2016

QuLogic added this to the 2.1 (next point release) milestone Mar 21, 2016

@QuLogic QuLogic added Documentation Rcparams labels Mar 21, 2016

madphysicist deleted the madphysicist:patch-1 branch Mar 21, 2016

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