Skip to content

Updated whats new for nbagg and legend/patheffects docs. - #3560

Merged
tacaswell merged 1 commit into
matplotlib:v1.4.xfrom
pelson:nbagg_whats_new
Sep 30, 2014
Merged

tacaswell merged 1 commit into
matplotlib:v1.4.xfrom
pelson:nbagg_whats_new

Conversation

@pelson

@pelson pelson commented Sep 24, 2014

Copy link
Copy Markdown
Member

I've put this against 1.4.x rather than 1.4.0-doc since the v1.4.1 release is so imminent, but I'll happily change if needs be.

@tacaswell tacaswell added this to the v1.4.1 milestone Sep 24, 2014
Comment thread doc/users/whats_new.rst

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably explicitly say that an explicit call to plt.show() is required (because we don't yet have the hooks set up right with ipython). See discussion at jupyter/docker-notebook#7 (comment)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably explicitly say that an explicit call to plt.show() is required

It isn't though - it behaves as any other backend does, namely, if you want to show a figure when not in interactive mode you have to do a plt.show(). I was also careful with the wording, in that it does currently state ...then subsequently shown.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Idea: It would be cool if display(fig) did the show for us even in non-interactive mode though.

tacaswell added a commit that referenced this pull request Sep 30, 2014
DOC : Updated whats new for nbagg and legend/patheffects docs.
@tacaswell
tacaswell merged commit b2b5227 into matplotlib:v1.4.x Sep 30, 2014
@pelson
pelson deleted the nbagg_whats_new branch October 3, 2014 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants