search

njtierney / geotargets

Targets extensions for geospatial data
https://njtierney.github.io/geotargets/
Other
49 stars 4 forks source link

Naming geotargets gdal raster options #25

Closed njtierney closed 3 months ago

njtierney commented 3 months ago

Breaking this out of #19

Naming things is always a bit tricky, but following on from @Aariq 's suggestion on renaming some of the env vars, current thoughts are:

GEOTARGETS_GDAL_RASTER_DRIVER_NAME -- > GEOTARGETS_GDAL_RASTER_DRIVER

It's a small change but I think it's worthwhile to reduce typing where practical. On the GDAL website they are called "Raster drivers" too, so there's that?

brownag commented 3 months ago

Totally agree with dropping "_NAME" from the option name, that is a great suggestion

I also agree with your point RE: "CREATION_OPTIONS" (https://github.com/njtierney/geotargets/pull/19#discussion_r1524052254).

At a minimum we need to distinguish between options used to write/create a datasource vs. those used to read/open them.

njtierney commented 3 months ago

closed in #30