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

geotiff driver kwarg #160

Closed
gottacatchenall opened this issue Feb 24, 2023 · 2 comments · Fixed by #161
Closed

geotiff driver kwarg #160

gottacatchenall opened this issue Feb 24, 2023 · 2 comments · Fixed by #161

Comments

@gottacatchenall
Copy link
Member

gottacatchenall commented Feb 24, 2023

In #30 (comment) we discussed having an option to save with different geotiff drivers (needed for cloud-optimizer-geotiff (COG), see GEO-BON/bon-in-a-box-pipelines#40 (comment)).

I don't think this is currently implemented, but would be as simple as moving the argument to this call

driver = ArchGDAL.getdriver("MEM"),
to be a keyword arg to save that is passed onto _write_geotiff

@gottacatchenall gottacatchenall changed the title geotiff drive save geotiff driver kwarg Feb 24, 2023
@gottacatchenall
Copy link
Member Author

in fact it seems this is already implemented for the version of write_geotiff for multiple layers

layers::Vector{SimpleSDMPredictor{T}};
, just not for a single layer

@tpoisot
Copy link
Member

tpoisot commented Feb 24, 2023

Can you open a PR? I'll merge when I'm back at my computer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants