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

'getting started' vignette #55

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

'getting started' vignette #55

wants to merge 9 commits into from

Conversation

njtierney
Copy link
Owner

@njtierney njtierney commented Apr 19, 2024

Still need to fill this in with more content from issues.

For example

This will resolve #17

Copy link
Collaborator

@Aariq Aariq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a good start! I think eventually we should add an example of how to change file type and pass GDAL options, but that could be in a separate PR if you'd rather.

vignettes/using-terra.Rmd Outdated Show resolved Hide resolved
vignettes/using-terra.Rmd Outdated Show resolved Hide resolved
vignettes/using-terra.Rmd Outdated Show resolved Hide resolved
@Aariq
Copy link
Collaborator

Aariq commented Apr 25, 2024

Is eval=FALSE just so CRAN doesn't complain? It might be better to precompute the vignette instead.

@njtierney
Copy link
Owner Author

Good idea with precomputing!

@njtierney njtierney marked this pull request as draft May 10, 2024 08:10
@Aariq Aariq changed the title get the vignette outline for tar_terra_ started. 'getting started' vignette Nov 15, 2024
@Aariq Aariq marked this pull request as ready for review November 15, 2024 19:03
@Aariq
Copy link
Collaborator

Aariq commented Nov 15, 2024

Because of the trick to set root.dir to a temporary directory, you don't need tar_dir() at all I think. All examples have three chunks: one with echo: false to write the _targets.R file with tar_script(), one with eval: false to show the contents of _targets.R, and one with echo and eval true to run the pipeline and inspect results.

I didn't include much about the raster and vector GDAL options, because I don't have a good motivating example, but I did include examples of changing the driver and added an example with preserve_metadata = "zip".

@Aariq
Copy link
Collaborator

Aariq commented Nov 18, 2024

@njtierney, I think this is pretty much done. Can you take a look when you get a chance?

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

Successfully merging this pull request may close these issues.

Create vignette for tar_terra_* targets use
2 participants