Skip to content

Create tar_terra_svc() for SpatVectorCollection objects #79

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

Open
Aariq opened this issue Jun 4, 2024 · 3 comments
Open

Create tar_terra_svc() for SpatVectorCollection objects #79

Aariq opened this issue Jun 4, 2024 · 3 comments
Labels
terra related to compatibility with `terra`

Comments

@Aariq
Copy link
Collaborator

Aariq commented Jun 4, 2024

I think at a minimum we should add an analog for SpatVectorCollection (i.e. tar_terra_svc() for terra::svc() results)

My initial response in the case of lists, which can be nested and heterogeneous, is that this may be a bit difficult to handle in general for geotargets methods.

However I can totally see the utility of list columns within a data.frame being a useful way to manage multiple terra objects and their metadata... so this is perhaps worth considering more.

Originally posted by @brownag in #77 (comment)

@Aariq Aariq added the terra related to compatibility with `terra` label Jul 22, 2024
@Aariq
Copy link
Collaborator Author

Aariq commented Sep 11, 2024

This will be more difficult to implement because:

  1. the equivalent to sub-datasets appears to be layers, which seem to be only supported by a limited number of formats (see insert arg in ?writeVector)

  2. When sds() is used on a file with multiple layers (saved using the insert arg), it can only read a single layer apparently (???)

@Aariq
Copy link
Collaborator Author

Aariq commented Sep 11, 2024

The reason I'm looking at this is because I'm thinking of possibly converting tar_terra_sprc(), tar_terra_sds(), and tar_terra_svc() into something like tar_terra_collection(name, command, type = c("sprc", "sds", "svc").

@Aariq Aariq added this to the rOpenSci submission milestone Oct 8, 2024
@Aariq Aariq removed this from the rOpenSci submission prep milestone Oct 31, 2024
@Aariq
Copy link
Collaborator Author

Aariq commented Oct 31, 2024

This seems tricky to implement so I think I'll hold off on this until after rOpenSci submission or if there's some user need for it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
terra related to compatibility with `terra`
Projects
None yet
Development

No branches or pull requests

1 participant