Skip to content
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

Nbconvert 1.0.0a1 does not take into account the pdf extensions in graphs #3735

Closed
jmgc opened this issue Jul 23, 2013 · 7 comments
Closed
Milestone

Comments

@jmgc
Copy link

jmgc commented Jul 23, 2013

I have prepared a report with one of the first versions of nbconvert. To have better graphics quality, I included the option in the notebook:

%config InlineBackend.figure_format = ’svg’

As a result, all the plots are in svg. When I transform to latex, the svg are converted to pdf without problems, but the latex command "\includegraphics" only has the name of the file, it does not include the 'pdf' extension.

On the other hand, would it be possible to convert to pdf all the svg files, also does included through markdown, this will simplify the work in notebook, as svg files will be available in the browser, and further translation to latex which does not "understand" svg.

Thanks for all your work,

Jose M.

@ivanov
Copy link
Member

ivanov commented Jul 23, 2013

Thanks for your report @jmgc. I see this is your first contribution on GitHub, so welcome to the party! 🍰

I'm preparing a fix for this if you want to give a try, it's #3736

@ivanov
Copy link
Member

ivanov commented Jul 24, 2013

pinging @jdfreder who's got more nbconvert involvement right now

@ivanov ivanov closed this as completed in 7b7cf49 Jul 25, 2013
minrk added a commit that referenced this issue Jul 25, 2013
Don't strip pdf extension from figure filenames

It seems to have been an erroneous assumption that this was necessary.

closes #3735
@jmgc
Copy link
Author

jmgc commented Jul 25, 2013

Why has been the present issue closed, taking into account that the problem is still there?

Thanks in advance

@ivanov ivanov reopened this Jul 25, 2013
@ivanov
Copy link
Member

ivanov commented Jul 25, 2013

I'm reopening it - this was just a side effect of some code that got merged which I originally commented was a possible fix for your issue.

back to the discussion that I tried continuing on #3736: how are you rendering the latex?

@jmgc
Copy link
Author

jmgc commented Jul 25, 2013

Presently I am using pdflatex from the MacTeX distribution. I should add that with the old stand-alone version of nbconvert things worked perfectly well, and with this version the ".pdf" was still there, and the names had all dots changed to "_", apart from the one in the ".pdf"

pdfTeX 3.1415926-2.5-1.40.14 (TeX Live 2013)
kpathsea version 6.1.1
Copyright 2013 Peter Breitenlohner (eTeX)/Han The Thanh (pdfTeX).
There is NO warranty. Redistribution of this software is
covered by the terms of both the pdfTeX copyright and
the Lesser GNU General Public License.
For more information about these matters, see the file
named COPYING and the pdfTeX source.
Primary author of pdfTeX: Peter Breitenlohner (eTeX)/Han The Thanh (pdfTeX).
Compiled with libpng 1.5.16; using libpng 1.5.16
Compiled with zlib 1.2.7; using zlib 1.2.7
Compiled with xpdf version 3.03

@ellisonbg
Copy link
Member

We are closing this as it is not an IPython issue.

@jmgc
Copy link
Author

jmgc commented Jul 27, 2013

Sincerely, I do not understand why you say it is not a problem of iPython. In my humble opinion, if a file from Nbconverter cannot be properly processed by LaTeX, is a problem from iPython. So please, can you tell me why this is not an iPython problem?

Thanks in advance

El 27/07/2013, a las 03:02, "Brian E. Granger" notifications@github.com escribió:

We are closing this as it is not an IPython issue.


Reply to this email directly or view it on GitHub.

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014
an alternative to this would be to do something like
`\DeclareGraphicsExtensions{.pdf}` in the preamble
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014
Don't strip pdf extension from figure filenames

It seems to have been an erroneous assumption that this was necessary.

closes ipython#3735
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants