Opened 6 years ago
Last modified 21 months ago
#17939 new defect
SageTex sageplot giving oversized plots with default Debian pdflatex.
Reported by: | anton_t | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | sage-6.6 |
Component: | graphics | Keywords: | sagetex latex pdflatex debian |
Cc: | tmonteil, vdelecroix, dimpase, chapoton | Merged in: | |
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
As above.
After using the third possible way of getting latex to talk to sagetex from here: http://www.sagemath.org/doc/installation/sagetex.html
The example code inside the sagetex folder produces a very ugly set of plots roughly twice the size of those shown here when : http://ctan.unsw.edu.au/macros/latex/contrib/sagetex/example.pdf
Using: Sage Version 6.5, Release Date: 2015-02-17 from github. pdfTeX 3.14159265-2.6-1.40.15 (TeX Live 2015/dev/Debian)
If no one else can reproduce this but I will attach the produced pdf. This however also happened under regular latex in Debian.
Change History (6)
comment:1 Changed 6 years ago by
comment:2 Changed 5 years ago by
- Cc tmonteil vdelecroix added
- Keywords debian added
comment:3 Changed 5 years ago by
Can you verify that this is only with Debian?
comment:4 Changed 4 years ago by
See also #22499 which might make it easier to deal with such tickets.
comment:5 Changed 2 years ago by
- Cc dimpase chapoton added
- Priority changed from major to minor
I seem to recall default image sizes getting a lot bigger a few years ago, but haven't seen any specific validation of this ticket. Certainly downgrading status as it is fairly easy (if tedious) to fix on an image-by-image basis.
comment:6 Changed 21 months ago by
t
is the list of matched tokens in a pyparsing
result. So in principle, t.opts
is the stuff in the first matching square brackets (if present) from a \sageplot
command.
However, if you are right then maybe we need to change
sageplotparser = (r'\sageplot' + Optional(squarebrackets)('opts') + Optional(squarebrackets)('format') + curlybrackets('code'))
to include the default in this part, rather than in the plot method
def plot(self, s, l, t): self.plotn += 1 if len(t.opts) == 0: opts = r'[width=.75\textwidth]' else: opts = t.opts[0] return (r'\includegraphics%s{sage-plots-for-%s.tex/plot-%s}' % (opts, self.fn, self.plotn - 1))
I'd want to test whether this is really being skipped but at any rate somewhere in here is where the fix would have to lie. Once we figure out whether this is an actual bug, we should report upstream.
By the way, the scripts dtx file is eminently readable!
After some minor digging around it looks like line 97 of sagetexparse.py never evaluates to true and
is never run. Since the name of the variable is "t" and I'm not familiar with the code base around those parts I'm not too keep on trying to fix it myself, as just trying to see where else in the code 't' appears made my eyes bleed.