-
Notifications
You must be signed in to change notification settings - Fork 0
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 COG processing for large images #891
Conversation
76748d3
to
91a1ab0
Compare
91a1ab0
to
b52745d
Compare
7d51f76
to
889f4f6
Compare
stripped_blob_stream.seek(0) | ||
accession_blob = self._generate_blob_as_cog(img) | ||
else: | ||
accession_blob = self._generate_blob(img) |
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.
Needs better function naming.
isic/ingest/models/accession.py
Outdated
is_cog=True, | ||
) | ||
self.blob = accession_blob.blob | ||
self._accession_blob_field.pre_save(self, add=False) |
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.
Can this be avoided?
self.blob = accession_blob.blob | ||
self._accession_blob_field.pre_save(self, add=False) | ||
|
||
return accession_blob |
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.
@brianhelba look at this
return accession_blob | |
yield accession_blob |
a4f4f50
to
8d26809
Compare
5d1f27b
to
1c480d5
Compare
1c480d5
to
818f033
Compare
related github.com/ImageMarkup/tracker/issues/123