-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Pass options to ps2pdf using -foo#bar
instead of -foo=bar
.
#10295
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
Conversation
"-dAutoFilterColorImages#false", | ||
"-dAutoFilterGrayImages#false", | ||
"-sGrayImageFilter#FlateEncode", | ||
"-sColorImageFilter#FlateEncode", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.... add the autorotate thing and I'll close my PR if this works...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-dAutoRotatePages#false
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
The former works both on Windows and Linux (so we can stick to it) whereas the later only works on Linux (so we had to use a switch depending on the OS).
37d4571
to
4ba9be1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This worked on my machine w/ cde
import matplotlib
matplotlib.rcParams['ps.usedistiller'] = 'xpdf'
import matplotlib.pyplot as pl
f = pl.figure (figsize=(7,5))
ax = f.add_axes ([0.15,0.15,0.8,0.8])
ax.set_ylim (0., 1.)
ax.set_xlim (0., 1.)
ntext = 8 # this is the minimum number of calls to get rotation
ytext = 0.5
for i in range (ntext) :
xtext = (1. / ntext) * i
n = 'test{}'.format (i)
ax.text (xtext, ytext, n, rotation='vertical')
f.savefig ('test.eps')
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pending CI...
Well it works on linux, and it should be the same code on Windows, so I'll take a chance and merge it. |
The former works both on Windows and Linux (so we can stick to it)
whereas the later only works on Linux (so we had to use a switch
depending on the OS).
Will conflict #10294, but easy to resolve either...
Closes #10290
PR Summary
PR Checklist