-
Notifications
You must be signed in to change notification settings - Fork 28
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
ENHANCE: Add Docker file for Jupyter support #248
Conversation
ScottyB
commented
Nov 15, 2020
- Adds a Docker file to the generated Surround projects and simplifies to doit commands for interacting with the notebook.
- Updated the generated notebook to load data from a runner and load the config file.
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.
web Dodo.py is still using the old tasks, that needs to be updated as well
'task_dep': ['build'], | ||
'params': PARAMS | ||
}} | ||
|
||
def task_jupyter(): |
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.
Update this for web version as well
8045e72
to
e4cb341
Compare
/test |
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.
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.
As discussed on slack, tornado issue will be fixed on a separate PR