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

Use rio-cogeo internally #138

Open
dionhaefner opened this issue Apr 3, 2019 · 7 comments
Open

Use rio-cogeo internally #138

dionhaefner opened this issue Apr 3, 2019 · 7 comments
Labels
enhancement New feature or request

Comments

@dionhaefner
Copy link
Collaborator

Currently, we have our own functions for optimizing and validating COG, which are more or less copied from rio-cogeo.

I did this because rio-cogeo did not feel quite mature enough at the time, but I think it might be time to re-evaluate.

@dionhaefner dionhaefner added the enhancement New feature or request label Apr 3, 2019
@vincentsarago
Copy link

👋 @dionhaefner I'll be more than happy to see this happening, feel free to give any feedback on the current implementation.
I was planning to do the 1.0.0 release today but I don't think I'll have time and we might ship cogeotiff/rio-cogeo#62 in 1.0.0

The only big difference is that rio-cogeo force everything to be done in-memory while you set up something to handle it differently cogeotiff/rio-cogeo#21

@dionhaefner
Copy link
Collaborator Author

Sure, we'd be happy to work with you to make this happen!

I think out-of-core operation will be a hard requirement for us as we often deal with enormous rasters. When we have some capacities we could look into merging that into rio-cogeo?

@mrpgraae
Copy link
Collaborator

mrpgraae commented Apr 4, 2019

Out-of-core operation is indeed a hard requirement for us. But yes, let's see if we can get that into rio-cogeo and use that from then on, that would be great for everyone.

Don't know how your time is looking @dionhaefner, but we might have some capacity for this here in DHI-GRAS within the next 3-4 weeks or so.

@dionhaefner
Copy link
Collaborator Author

Don't know how your time is looking @dionhaefner, but we might have some capacity for this here in DHI-GRAS within the next 3-4 weeks or so.

That's great, since my bandwidth is going to be limited for a while :)

@dionhaefner
Copy link
Collaborator Author

@vincentsarago Could you run me through what you are hoping to achieve with the web optimization (cogeotiff/rio-cogeo#62)? I think it looks very interesting, but I'm not sure where the actual advantages are / what your motivation is. Are you hoping for better performance (e.g. for a tool like Terracotta), or do you primarily want to avoid reprojection artifacts?

@vincentsarago
Copy link

sure @dionhaefner The idea is to have a COG aligned on the web mercator grid and zoom levels corresponding to overview levels. The main purpose is to reduce the number of internal tiles fetched to the strict minimum.

avoid reprojection artifacts

Those will in fact happen when creating the COG.

TBH I'm not really planning to use this options but I can see how it could be a nice feature when dealing with mosaics or global dataset.

@dionhaefner
Copy link
Collaborator Author

This could be amazing for Terracotta. We have had something similar to this in the past, but I couldn't get it to work right when retrieving the tiles because boundless windowed reads didn't work properly with rasterio (so we had to revert to using a VRT). Would be interesting to see whether it makes a big difference performance-wise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants