Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Subplot spacing#6468

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

Merged
efiring merged 3 commits intomatplotlib:v2.xfromtacaswell:subplot-spacing
May 28, 2016
Merged

Subplot spacing#6468

efiring merged 3 commits intomatplotlib:v2.xfromtacaswell:subplot-spacing
May 28, 2016

Conversation

tacaswell
Copy link
Member

Replaces#6129

@efiring
Copy link
Member

This has a bunch of image failures. Some of them I recognize as trouble-makers from earlier style changes. I'm not sure whether the best solution is just replacing the images.

@tacaswell
Copy link
MemberAuthor

Digging into why these are failing is on my list of things to do today.

On Sat, May 28, 2016, 14:03 Eric Firingnotifications@github.com wrote:

This has a bunch of image failures. Some of them I recognize as
trouble-makers from earlier style changes. I'm not sure whether the best
solution is just replacing the images.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#6468 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAMMhcBesWznV5Un9AA24LbBtbk09OAWks5qGINmgaJpZM4IkzRk
.

@tacaswell
Copy link
MemberAuthor

3 of the tests that explicitly use the new defaults and 4 of them are the pgf tests.

I think that until we tag 2.0, the image comparison should just knownfail any non-'classic' style test.

@tacaswell
Copy link
MemberAuthor

Assuming this passes it should be ready to go. The other default change PRs should probably be rebased to also side-step some of this pain.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging this pull request may close these issues.

3 participants
@tacaswell@efiring@mdboom

[8]ページ先頭

©2009-2025 Movatter.jp