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

Create and Update Dockerfile #143

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tbeckenhauer
Copy link

It has a few additional helper scripts.

@tbeckenhauer
Copy link
Author

@miku86 FYI. This was mainly an experiment for me to learn how to mirror users and map the directories to try to make things as streamlined as possible.

@stackcoder
Copy link

You might also add the neuronal network extractor_2.h5 file to the container. So it does not download 162MB every time you run it. The web server hosting it is not as fast as GitHub oder Docker Hub would be.

Also I would retain the Dockerfile free from hardcoded user ids. These are depend to the Docker host and can be simply applied at runtime.

I created another Dockerfile here: https://gist.github.com/stackcoder/584ec03d63163eb9d550df7ac8758ef7

@tbeckenhauer
Copy link
Author

@stackcoder Excellent point about the neuronal network file. Ill update it.

Also, Those user IDs will get changed when you run the start script. They are just placeholders for the build process. They ultimately get replaced with your normal users IDs so any files created are accessible by your user account.

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.

None yet

3 participants