/matplotlib

ENH: Added `file` keyword to `setp` to redirect output #6222

Merged
merged 1 commit into from Mar 26, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  2.1 (next point r...
3 participants
@madphysicist @tacaswell @mdboom
@madphysicist
Contributor

madphysicist commented Mar 24, 2016

Added this based on something I noticed with #6217.

mdboom added the needs_review label Mar 24, 2016

@madphysicist
Contributor

madphysicist commented Mar 25, 2016

Failure is unrelated to PR. Image mismatch.

tacaswell added this to the 2.1 (next point release) milestone Mar 25, 2016

@tacaswell
Owner

tacaswell commented Mar 26, 2016

This might need an entry in https://github.com/matplotlib/matplotlib/tree/master/doc/users/whats_new

What is the motivation for this?
@madphysicist
Contributor

madphysicist commented Mar 26, 2016

I wanted to be able to print to sys.stderr mostly. My stdout was going to a file. This seemed like the easiest thing to do at the time.
@madphysicist @madphysicist madphysicist ENH: Added `file` keyword to `setp` to redirect output

Some checks were not successful

1 failing and 1 successful checks
continuous-integration/appveyor/pr — AppVeyor build failed
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
0b56a6a
@madphysicist
Contributor

madphysicist commented Mar 26, 2016

Rebased and added a doc to whats_new.

@tacaswell tacaswell merged commit c06bb2b into matplotlib:master Mar 26, 2016
1 of 2 checks passed

1 of 2 checks passed

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

tacaswell removed the needs_review label Mar 26, 2016

madphysicist deleted the madphysicist:setp-output branch Mar 26, 2016

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