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

Changesavefig.dpi andfigure.dpi defaults#5659

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 1 commit intomatplotlib:masterfromjmehne:fix_305_2
Dec 13, 2015

Conversation

jmehne
Copy link
Contributor

Change the default values forsavefig.dpi andfigure.dpi as suggested in#305.

savefig.dpi: 100 -> 'figure'figure.dpi: 80 -> 100

@mdboommdboom added this to thenext major release (2.0) milestoneDec 11, 2015
@tacaswell
Copy link
Member

👍 from me

efiring added a commit that referenced this pull requestDec 13, 2015
Change `savefig.dpi` and `figure.dpi` defaults
@efiringefiring merged commit6333526 intomatplotlib:masterDec 13, 2015
@efiring
Copy link
Member

Cherry-picked to v2.0.x asa903315.@tacaswell, did I do that correctly?

@mdboom
Copy link
Member

@efiring: We've been cherry-picking the merge commit (which in this case is6333526) rather than the individual commits (in this case016406a). I think you did the latter. It's not a huge difference, but it does keep a mention in the logs of which PR a feature came from for later reference if you do the former. It's also easier to merge a multi-commit PR that way.

@efiring
Copy link
Member

@mdboom, OK, that makes sense, thank you. That requires using the "-m parent" option to the "git cherry-pick" command, doesn't it? How does one know the parent number?

@tacaswell
Copy link
Member

It is always parent 1 because we are meeting things into master

On Sun, Dec 13, 2015, 10:33 Eric Firingnotifications@github.com wrote:

@mdboomhttps://github.com/mdboom, OK, that makes sense, thank you.
That requires using the "-m parent" option to the "git cherry-pick"
command, doesn't it? How does one know the parent number?


Reply to this email directly or view it on GitHub
#5659 (comment)
.

@efiring
Copy link
Member

Travis is complaining about the original commit, but it seems unrelated to the actual file changes. Should I be worried?

ERROR: matplotlib.tests.test_axes.test_loglog.test----------------------------------------------------------------------Traceback (most recent call last):  File "/home/travis/build/matplotlib/matplotlib/venv/lib/python3.5/site-packages/nose/case.py", line 198, in runTest    self.test(*self.arg)  File "/home/travis/build/matplotlib/matplotlib/venv/lib/python3.5/site-packages/matplotlib-1.5.0+127.ga903315-py3.5-linux-x86_64.egg/matplotlib/testing/decorators.py", line 54, in failer    result = f(*args, **kwargs)  File "/home/travis/build/matplotlib/matplotlib/venv/lib/python3.5/site-packages/matplotlib-1.5.0+127.ga903315-py3.5-linux-x86_64.egg/matplotlib/testing/decorators.py", line 235, in do_test    self._tol, in_decorator=True)  File "/home/travis/build/matplotlib/matplotlib/venv/lib/python3.5/site-packages/matplotlib-1.5.0+127.ga903315-py3.5-linux-x86_64.egg/matplotlib/testing/compare.py", line 306, in compare_images    raise IOError('Baseline image %r does not exist.' % expected)OSError: Baseline image '/home/travis/build/matplotlib/matplotlib/result_images/test_axes/loglog-expected.png' does not exist.

@tacaswell
Copy link
Member

Something seems to have gone wrong with that file,#5652 had the same issue.

@jmehne
Copy link
ContributorAuthor

Hm, strange. It seems like only AppVeyor was run.

@mdboommdboom mentioned this pull requestDec 22, 2015
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging this pull request may close these issues.

4 participants
@jmehne@tacaswell@efiring@mdboom

[8]ページ先頭

©2009-2025 Movatter.jp