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

Add link to Deepnote #332

Merged
merged 5 commits into from
Aug 6, 2019
Merged

Add link to Deepnote #332

merged 5 commits into from
Aug 6, 2019

Conversation

jirkalhotka
Copy link
Contributor

I like Deepnote more than Jupyter's nbviewer, because it allows you to edit and save the notebooks online as well. I added a link to the instructions list in case anyone else wanted to play around with the code as well.

@jirkalhotka
Copy link
Contributor Author

jirkalhotka commented Dec 18, 2018

Similar service to Mybinder from #190, but more stable and a bit faster 🤗

@ageron
Copy link
Owner

ageron commented Dec 19, 2018

Hi @Jirka-Lhotka ,
Thanks for your proposal. I did not know about Deepnote. How stable is it? I just tried clicking on the link for chapter 1, and after a few seconds, Deepnote printed an error message "Could not connect to orchestrator, received 502". I had a similar experience with mybinder: I added a link to it almost at the very beginning of this project, and I really thought it was fantastic, but I ended up removing it because of the fact that it would regularly be unavailable or buggy, and people would file issues here.

@jirkalhotka
Copy link
Contributor Author

Hey @ageron, thanks for the prompt response! I switched to Deepnote from mybinder exactly for this reason and haven't really had any problems -- sometimes a library is missing, but generally it has all the main ones. I've experienced the 502 error once, but it fixed itself when I clicked the link again so it wasn't a big issue really.

@jirkalhotka
Copy link
Contributor Author

Hi @ageron, we've done a lot of work on Deepnote to make it more stable. I think it might make more sense for you now. Check it out and let me know!:)

@ageron
Copy link
Owner

ageron commented Aug 4, 2019

Hi @Jirka-Lhotka ,
Cool! I've updated the README for the 2nd edition to point to Deepnote, as well as a few other options.

@jirkalhotka
Copy link
Contributor Author

jirkalhotka commented Aug 6, 2019

Great to hear @ageron, thanks for that! And also thanks for pointing out the bug with requirements.txt not being installed automatically; we're working on fixing that!

What are your thoughts on merging this PR? I understand the 2nd version of the book is what you want to push going forward, but I think the readers of the 1st ed could enjoy Deepnote too and I'd love to get some more users on there:)

@ageron
Copy link
Owner

ageron commented Aug 6, 2019

Hi @Jirka-Lhotka ,
Sounds good, thanks for the PR! :)

@ageron ageron merged commit 7de3285 into ageron:master Aug 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants