-
Notifications
You must be signed in to change notification settings - Fork 5
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
dockerize repo #11
Comments
will do after implementing Jim's code framework |
Do you have a scope that you want dockerize here ? |
What do you mean scope? I was thinking since we'd like to eventually have this repo used widely, we'd want to container-ize it so that all the dependencies are wrapped up. Another challenge to solve is figuring out where to host and then access the full dataset. This will depend on the host. |
@ngiangre I mean should we only containerize the Dash app itself or we need to include environment jupyter notebooks as well ? |
Hmmm I would think both the notebooks and app would be a package deal. Do you have a suggestion here?
Cheers,
Nick
From: hsiao yi <[email protected]>
Reply-To: NCBI-Hackathons/SafeDrugs <[email protected]>
Date: Thursday, August 16, 2018 at 11:38 AM
To: NCBI-Hackathons/SafeDrugs <[email protected]>
Cc: Nick Giangreco <[email protected]>, Mention <[email protected]>
Subject: Re: [NCBI-Hackathons/SafeDrugs] dockerize repo (#11)
@ngiangre I mean should we only containerize the Dash app itself or we need to include environment jupyter notebooks as well ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I think we can have two separate docker containers for these two different parts. One for main Dash app and one for jupyter notebook. Eventually, all plots presented in jupyter notebooks would be moved integrated into Dash app, right ? |
Ideally, yes. That was the thought process initially. Have the notebooks be used by one audience, and the app used by an overlapping audience where the notebook code transfers “seamlessly” with the app’s code.
Cheers,
Nick
From: hsiao yi <[email protected]>
Reply-To: NCBI-Hackathons/SafeDrugs <[email protected]>
Date: Thursday, August 16, 2018 at 12:26 PM
To: NCBI-Hackathons/SafeDrugs <[email protected]>
Cc: Nick Giangreco <[email protected]>, Mention <[email protected]>
Subject: Re: [NCBI-Hackathons/SafeDrugs] dockerize repo (#11)
I think we can have two separate docker containers for these two different parts. One for main Dash app and one for jupyter notebook. Eventually, all plots presented in jupyter notebooks would be moved integrated into Dash app, right ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
The data is public and yes I'd want to put it in the public domain eventually |
No description provided.
The text was updated successfully, but these errors were encountered: