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

DefaultTileStore: should ignore .tmp files #125

Open
favyen2 opened this issue Jan 16, 2025 · 0 comments
Open

DefaultTileStore: should ignore .tmp files #125

favyen2 opened this issue Jan 16, 2025 · 0 comments

Comments

@favyen2
Copy link
Collaborator

favyen2 commented Jan 16, 2025

get_raster_bounds throws error if there are multiple non-completed files in the raster directory. But this situation can arise if an ingest job fails, leaving a temporary file (due to the atomic write code), and then a later ingest job succeeds. We should ignore .tmp files in get_raster_bounds and read_raster. Maybe we can write a test that passes a custom UPath to write_raster_file which fails halfway, which should yield a .tmp file, and then write a normal raster file, and then try to read it?

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

No branches or pull requests

1 participant