-
Notifications
You must be signed in to change notification settings - Fork 571
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
Slides broken? #344
Comments
I don't think it's broken, but it does by default use reveal.js from a CDN. Is it possible that you were offline when you did this? Or maybe the CDN was down. There is an option to use a local copy of reveal instead. |
Not sure if it's significant, but I get a 403 forbidden from the cdn url: Is anyone else seeing that or is it just me? I'm behind a corporate firewall but I have my I'll try a local reveal to see if that works for me... |
A local install of reveal did work. It would be nice for the cdn to work so if others are also seeing problems with it this issue could be kept open. If it was just something specific to my particular setup and is working for others I'm happy for this to be closed... |
From my browser I can navigate to https://cdnjs.cloudflare.com/ajax/libs/reveal.js/3.3.0/js/reveal.min.js without error so I thought I would try to override the default reveal cdn location using
|
It seems the 403 might be a red-herring as the full url https://cdnjs.cloudflare.com/ajax/libs/reveal.js/3.1.0/js/reveal.js works fine so I assume nbconvert adds the Specifying At this stage I'm thinking its possibly something to do with the proxy in which case, as mentioned, I'm happy for this to be closed. I'll try and repro when I'm not behind a proxy to verify but that probably won't happen until next week... |
It does add e.g. |
I have a similar issue, which I (wrongly) posted here in the |
@mghijs did the local install of reveal work for you? |
@michaelpacer thanks for the suggestion, it didn't work either though and gave me this error message: jupyter nbconvert general_info.ipynb --to slides --post serve
[NbConvertApp] Converting notebook general_info.ipynb to slides
[NbConvertApp] Writing 392021 bytes to general_info.slides.html
[NbConvertApp] Serving local reveal.js
Serving your slides at http://127.0.0.1:8000/general_info.slides.html
Use Control-C to stop this server
WARNING:tornado.access:404 GET /custom.css (127.0.0.1) 3.00ms
WARNING:tornado.access:404 GET /custom.css (127.0.0.1) 1.00ms
WARNING:tornado.access:404 GET /favicon.ico (127.0.0.1) 1.00ms |
One more question @mghijs, are you running from master or last release? |
I was running from master, but just tried the reveal.js-3.3.0 release and that gave the same result. (including the same error message) |
@dhirschfeld, I tried with latest nbconvert (5.2.1) and I can not reproduce your issue. Can you test it again with latest released version and see if you still see the issue? @mghijs, those 404 for custom.css are not harmful, can you also try with latest nbconvert and see if you still see the issue? Thanks. |
I could not replicate this issue, so I will close it now. |
I have the issue as @mghijs had on Aug 8 2016. I used nbconvert on my Win10 machine but not on my Linux virtual box where the problem didnt happen. On the Linux machine I do not get the favicon error but on the windows machine I do. On my Win10 machine I use 64bit OS and miniconda. Both machines run nbconvert 5.2.1. @damianavila I hope this can help you reproduce the error. |
When I try to create a reveal slideshow i get a whole bunch of errors thrown and it doesn't display properly. Is the slideshow functionality broken?
I've uploaded my test notebook to: https://anaconda.org/dhirschfeld/test_presentation/notebook
The text was updated successfully, but these errors were encountered: