search

njtierney / geotargets

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

Rename tar_<filetype>() to tar_<pkg>_<data type>() #7

Closed Aariq closed 6 months ago

Aariq commented 7 months ago

This PR renames tar_geotiff() to tar_terra_rast(). It separates out the format into a new function, format_terra_rast() that returns the output of a tar_format() call. I separated it out so it could accept a filetype argument where users can choose a GDAL driver (from a list of possibilities we have implemented and tested with targets).

I'm thinking maybe we'll end up having tar functions like `tar_<data type/class>()` with some default storage formats rather than naming the functions after the file type. What do you think?

Somewhat addresses #4

njtierney commented 6 months ago

I really like the idea of having tar_<pkg name>_<data type/class>(). I think this will help encourage discoverability for users. Nice one!

Aariq commented 6 months ago

Looks good! Thanks!