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

Added dependencies for workshop #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

fgreg
Copy link

@fgreg fgreg commented Jul 10, 2018

@kmaull-ucar I added the steps to install our dependencies as you suggested. However when I try to build the image it fails this install:

conda install -y -q -c conda-forge hone

With this error:

PackagesNotFoundError: The following packages are not available from current channels:

  • hone

Current channels:

To search for alternate channels that may provide the conda package you're
looking for, navigate to

https://anaconda.org

and use the search bar at the top of the page.

But if I just build the image with my instructions it works. Maybe you can try it?

@scgordon
Copy link
Collaborator

That's my fault. I just assumed it was available from anaconda as well as pypi. It is not.

We can drop hone. I can use the pip install it as an example of how people can customize their environment beyond the base. It's for a bonus notebook anyway and non essential even then.

@fgreg
Copy link
Author

fgreg commented Jul 13, 2018

@scgordon No worries, can we get this merged so it gets in the docker image?

@scgordon
Copy link
Collaborator

Sorry Frank, I thought they got merged when hone did. I'll investigate after my morning meeting, and get a new build with your requirements going.

@craig-willis
Copy link
Contributor

@fgreg @scgordon I've merged a separate PR #19 that includes these changes. I restructured the Dockerfile a bit and fixed some small issues. This image should be available now via autobuild (https://hub.docker.com/r/ndslabs/esiphub-notebook/builds/). Let me know if I missed anything.

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.

3 participants