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 README.md #89
Update README.md #89
Conversation
The docker base image you're using is
Is this something that should even be included in the docs? It's a useful hack for development, but not necessary for using it. |
Okay, the scripts are actually located in /site-packages, not /dist-packages. Maybe because of how it's installed? Happy to remove if you don't think it's necessary |
Awesome, let me know what you think of the new version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some very minor changes/suggestions.
Looks good. The only thing I think should be added is a description of our ontology. But that is covered as outstanding in #57 |
Updates the readme to reflect the current workflow. Let me know what areas could use further improvement.
@willgraf, one question for you. The current code to mount the codebase doesn't actually work.
-v $PWD/caliban_toolbox:/usr/local/lib/python3.5/dist-packages/caliban_toolbox/
The instructions were probably copied from deepcell-tf. However, in this docker there's nothing inside dist-packages, so mounting it does nothing.
Is there somewhere else I should be looking for docker's version of the scripts?