-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
hatch linewidth patch #235
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I can't access the patch: clicking on the "Original report" link shows a page displaying the error message "Error - The Tracker has been disabled for this group". |
@ddale: Do you know if we can still access the original SF issue? |
@jcmdev0 : Are you the original author of this. I am interested to know if you have still got the diff you provided. Sadly we lost it in the migration from sourceforge to github. If not, we will have to close this issue. Thanks. |
@pelson: I don't think I have the patch file any more. I think I remember enough of what it looked like to try to cook it up again real quick. |
I don't see a way for me to add a patch to this issue on github? I made a fork that has a hatch width change for the ps and pdf backends: https://github.com/jcmdev0/matplotlib |
Make a pull request from your fork, and in that pull request reference the present issue number as #235; that should automatically make a link from here to the PR. If it doesn't, you can refer to the pull request from here, and that will link the two. Also, if you haven't already, see http://matplotlib.org/devel/index.html. |
@efiriring, thanks! |
@jcmdev0 You can only have one pull request per branch. So you won't be able to submit another pull request while the changes on your master branch are still waiting to be merged. If, in the future, you'd like to have more than one pull request open at a time you can do the following:
This creates a separate branch for your changes. |
Now that @jcmdev0 has made the PR, I will close this obsolete issue. @jcmdev0 : Thank you ever so much for dragging this back into life. The move to GitHub has been a real good change, but it did have some negative side effects (this issue being a good example of that), so your work is greatly appreciated! |
Was this feature ever implemented ? |
I am guessing that the relevant pull request is #1200 Which was not merges so no it doesn't look like it |
Any chance of having this issue re-opened ? What stopped a merge last time (other than git issues) ? Just got bitten by this bug while creating a last minute figure for a deadline (had to edit exported SVG by hand !), |
If you would like to pick it up, that would be great! |
I just hit this problem. Has it really not been fixed yet? |
nope. the PR never made it in. |
Looking over the PR, it looks like tests were failing and the PR was On Wed, Nov 11, 2015 at 11:52 AM, Rodrigo Benenson <notifications@github.com
|
I'm going to reopen this for examination during the style changes. There's sort of two questions here:
I'd say resolving (1) is critical for 2.0 while we're changing other style things. (2) would also be really nice, but could be punted on if we can't get it done. |
Original report at SourceForge, opened Tue Feb 22 17:26:22 2011
The pdf backend uses a 0.1pt linewidth for hatch marks, which, depending on scaling and ultimate output, can make them impossible to see. I haven't looked at any non-pdf backends (pdf seems to be the only one that implements a writeHatches function), and I don't know if the rc var name is reasonable, but this seems like the beginning of a fix everyone can enjoy.
SourceForge History
The text was updated successfully, but these errors were encountered: