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

Add command to install deepcell spots in docker container #41

Merged
merged 2 commits into from
Feb 2, 2023

Conversation

msschwartz21
Copy link
Member

Currently deepcell-spots does not get installed in the docker container unless you mount the spots folder into the correct folder in the running container. This PR adds the installation to the Dockerfile so that the package is always available.

@msschwartz21 msschwartz21 added the fix Fixes a bug label Jan 31, 2023
Copy link
Contributor

@rossbar rossbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming the deepcell_spots from L24 is the one that defines the Python package (i.e. has the __init__.py in it) then this makes sense to me!

@elaubsch elaubsch merged commit c0fd598 into master Feb 2, 2023
@elaubsch elaubsch deleted the mrgn/dockerfile branch February 2, 2023 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants