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

Update and Add Binder #13

Merged
merged 2 commits into from Dec 26, 2017

Conversation

Projects
None yet
2 participants
@radovankavicky
Contributor

radovankavicky commented Dec 22, 2017

For those who don't want to install and use Docker (or avoid it).

Similarly like Jake has it here: https://github.com/jakevdp/PythonDataScienceHandbook

Binder simply builds a Docker image of your repository & will search for a dependency file, such as requirements.txt or environment.yml, in the repository's root directory and render all the notebooks for you.

+Maybe an index file could be added in future... but I will leave that for you (if you want it).

radovankavicky added some commits Dec 22, 2017

Update and Add Binder
For those who don't want to install and use Docker.
Add requirements.txt file (for Binder)
Here is a list of dependencies (you can modify it according to the notebooks and libraries there added)
@radovankavicky

This comment has been minimized.

Show comment
Hide comment
@radovankavicky

radovankavicky Dec 22, 2017

Contributor

Added also list of dependencies (you can modify it according to the notebooks and libraries there added)

Contributor

radovankavicky commented Dec 22, 2017

Added also list of dependencies (you can modify it according to the notebooks and libraries there added)

@sachinruk sachinruk merged commit 59cf2eb into sachinruk:master Dec 26, 2017

@sachinruk

This comment has been minimized.

Show comment
Hide comment
@sachinruk

sachinruk Dec 27, 2017

Owner

@radovankavicky Im actually getting the following error when I try to launch it. Any thoughts?

Image gcr.io/binder-prod/r2d-fd74043sachinruk-deepschool.io:59cf2ebc3fa166accc630e42a5e7af59ea334
673 for user sachinruk-deepschool.io-uicpr1kp took too long to launch
Owner

sachinruk commented Dec 27, 2017

@radovankavicky Im actually getting the following error when I try to launch it. Any thoughts?

Image gcr.io/binder-prod/r2d-fd74043sachinruk-deepschool.io:59cf2ebc3fa166accc630e42a5e7af59ea334
673 for user sachinruk-deepschool.io-uicpr1kp took too long to launch
@radovankavicky

This comment has been minimized.

Show comment
Hide comment
@radovankavicky

radovankavicky Jan 20, 2018

Contributor

I tried it again too & now taking a long time to load and nothing. After I clicked to see the logs for details (as suggested by Binder) and there was only:

Image gcr.io/binder-prod/r2d-fd74043sachinruk-deepschool.io:d3b02a5da91a2de56e3d0c39f8363d2a3b3fd137 for user sachinruk-deepschool.io-84v8w49c took too long to launch

Similar issue is described here: binder-project/binder#88

Maybe it is because of changes in repo: binder-project/binder#41

Or maybe we need to update requirements.txt document within the repo (to update list of dependencies).

Not sure how to resolve it (it was working only at the beginning/not after the changes in repository).

I wrote & described the issue on their Gitter (hopefully there will be somebody from the Binder developers who could help/point in the right direction how to resolve the issue).

Contributor

radovankavicky commented Jan 20, 2018

I tried it again too & now taking a long time to load and nothing. After I clicked to see the logs for details (as suggested by Binder) and there was only:

Image gcr.io/binder-prod/r2d-fd74043sachinruk-deepschool.io:d3b02a5da91a2de56e3d0c39f8363d2a3b3fd137 for user sachinruk-deepschool.io-84v8w49c took too long to launch

Similar issue is described here: binder-project/binder#88

Maybe it is because of changes in repo: binder-project/binder#41

Or maybe we need to update requirements.txt document within the repo (to update list of dependencies).

Not sure how to resolve it (it was working only at the beginning/not after the changes in repository).

I wrote & described the issue on their Gitter (hopefully there will be somebody from the Binder developers who could help/point in the right direction how to resolve the issue).

@radovankavicky

This comment has been minimized.

Show comment
Hide comment
@radovankavicky

radovankavicky Jan 21, 2018

Contributor

@sachinruk
So, here is how to resolve it (reaction directly from Binder developers/they are amazing... reacted very quickly, so let's try to do it the way they suggested):

Tim Head @betatim Jan 20 22:29
@radovankavicky you have a Dockerfile in your deepschool.io repo, so binder is trying to build that image and expects it to start a notebook server in a particular way
if you aren't using the dockerfile to build the binder image then I think the best way to get binder to ignore it is to create a directory called binder/ and place a requirements.txt in it that contains your dependencies
i think as soon as there is a binder/ subdirectory we ignore everything in the top level directory

Chris Holdgraf @choldgraf Jan 20 23:07
@radovankavicky I made a PR to do this, see my fork binder link here: https://mybinder.org/v2/gh/choldgraf/deepschool.io/binder_folder?filepath=notebooks%2FIndex.ipynb
however, I noticed you've got a bug in your requirements code (not your bug, but a scikit-image bug)
the last version of skimage depends on a version of numpy that has a bug in it, which means you won't be able to pip install -r with that skimage version

Contributor

radovankavicky commented Jan 21, 2018

@sachinruk
So, here is how to resolve it (reaction directly from Binder developers/they are amazing... reacted very quickly, so let's try to do it the way they suggested):

Tim Head @betatim Jan 20 22:29
@radovankavicky you have a Dockerfile in your deepschool.io repo, so binder is trying to build that image and expects it to start a notebook server in a particular way
if you aren't using the dockerfile to build the binder image then I think the best way to get binder to ignore it is to create a directory called binder/ and place a requirements.txt in it that contains your dependencies
i think as soon as there is a binder/ subdirectory we ignore everything in the top level directory

Chris Holdgraf @choldgraf Jan 20 23:07
@radovankavicky I made a PR to do this, see my fork binder link here: https://mybinder.org/v2/gh/choldgraf/deepschool.io/binder_folder?filepath=notebooks%2FIndex.ipynb
however, I noticed you've got a bug in your requirements code (not your bug, but a scikit-image bug)
the last version of skimage depends on a version of numpy that has a bug in it, which means you won't be able to pip install -r with that skimage version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment