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.

Sign up for GitHub

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

Jump to bottom

CXGN::Image process_image does not overwrite images with existing image id #24

Open
nickpalladino opened this issue Oct 30, 2019 · 1 comment
Open

CXGN::Image process_image does not overwrite images with existing image id #24

nickpalladino opened this issue Oct 30, 2019 · 1 comment

Comments

@nickpalladino
Copy link
Contributor

When you pass in an image id that exists, a warning is printed out saying that the image will be overwritten but then a new image id is created.
@lukasmueller
Copy link
Member

lukasmueller commented Nov 4, 2019 •
edited
Loading

I think this is the correct behavior, because images are stored tied to their md5sum, so it would be illogical to store a different image at that location. Maybe just the warning needs to be changed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lukasmueller @nickpalladino