Skip to content

Adjust default linewidth to 1.5 - #6286

Closed
efiring wants to merge 2 commits into
matplotlib:masterfrom
efiring:linewidth
Closed

efiring wants to merge 2 commits into
matplotlib:masterfrom
efiring:linewidth

Conversation

@efiring

@efiring efiring commented Apr 10, 2016

Copy link
Copy Markdown
Member

The default linewidth was changed from 1 to 2.5 for v2.0 in rcsetup, but was given as 2.0 in matplotlibrc.template. Although a fat line can look good for very simple plots, it does not work well for more complex plots. Therefore I recommend a compromise value of 1.5. Regardless of whether this recommendation is accepted, the value in rcsetup needs to match the value in matplotlibrc.template, as it does with this PR.

@efiring efiring added this to the 2.0 (style change major release) milestone Apr 10, 2016
@efiring

efiring commented Apr 10, 2016

Copy link
Copy Markdown
Member Author

It looks like the test failures are all image comparisons involving backend_pgf--that seems odd.

@QuLogic QuLogic added the API: default changes Changes to default behavior label Apr 11, 2016
@jenshnielsen

Copy link
Copy Markdown
Member

@efiring The pfg backend tests does the image comparison in a slightly odd way so it's not running with the classic style but with the default one

@mdboom

mdboom commented Apr 11, 2016

Copy link
Copy Markdown
Member

Yeah. 👍 on this (once the PGF baselines are updated).

@tacaswell

Copy link
Copy Markdown
Member

👍 on this from me as well.

@jenshnielsen

Copy link
Copy Markdown
Member

And me too 👍

@efiring

efiring commented Apr 12, 2016

Copy link
Copy Markdown
Member Author

Closing and reopening after merging #6290 to trigger the tests again.

@efiring

efiring commented Apr 12, 2016

Copy link
Copy Markdown
Member Author

Trying again after removing the commit that modified the pgf test files.

@efiring

efiring commented Apr 13, 2016

Copy link
Copy Markdown
Member Author

I'm stuck; after many tries, I still can't get a mergeable collection of changesets in which the pgf tests pass.

@mdboom

mdboom commented Apr 13, 2016

Copy link
Copy Markdown
Member

My guess is some mismatch in LaTeX libraries between your machine and Travis. I've had success in the past with this, so I'll give it a go.

mdboom added a commit that referenced this pull request Apr 14, 2016
mdboom added a commit that referenced this pull request Apr 14, 2016
@mdboom

mdboom commented Apr 14, 2016

Copy link
Copy Markdown
Member

Closed by #6300.

@efiring
efiring deleted the linewidth branch October 24, 2021 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants