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

First swing at ZipFile support. (Issue #16) #17

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JosephCatrambone
Copy link
Owner

These changes are a little messy. I think the IndexedImage file needs a better way to read from an in-memory image. The handling of zipfiles is also not great, and I think when we start monitoring for on-disk changes it's going to get worse. Perhaps the crawler's ingesting methods could be split off into a separate file.

WIP for #16

…ingest. Need a better method in indexed image for reading from in-mem images.
@JosephCatrambone
Copy link
Owner Author

Current change breaks things.

...
Error processing E:\GraphicsAssets\wood_patterned_21-color.jpg: The image format could not be determined
Error processing E:\GraphicsAssets\xoana_splash.png: The image format could not be determined
Error processing E:\GraphicsAssets\youwin.png: The image format could not be determined
...

@JosephCatrambone
Copy link
Owner Author

Problem was I was only doing buffer.read() instead of buffer.read_to_end(). The one thing that should probably get cleaned up before this is merged is the view screen, which expects that the file is readable from disk. If it comes in as a zipfile and we can't just open it, we might hit problems. Perhaps the indexedimage needs an extra field for 'image is stored as zip'?

…rom a zip we might know. Not very happy with this solution.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant