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

Updates for easier rendering (local and remote) #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

zachperiscopic
Copy link

Was running into trouble running the tool, turns out a unit parameter was not explicitly named in a terra::cellSize call, which led to an invalid type.

Other:

  • Update comment spacing of root dir due to keyboard shortcut commenting

@zachperiscopic zachperiscopic changed the title Explicit Unit parameter Fix: Add explicit 'unit' parameter Oct 3, 2023
@zachperiscopic zachperiscopic changed the title Fix: Add explicit 'unit' parameter Updates for easier rendering (local and remote) Oct 6, 2023
@zachperiscopic
Copy link
Author

zachperiscopic commented Oct 6, 2023

Fixing error: The error you reference in the user guide seems to originate partly from naming your files with spaces (used to be EFA markdown.Rmd). Updated the file names to fix this.

  • You should be able to render now locally without specifying/updating the root directory

Removed pacman: It makes it easier to install packages automatically, but made it tough when deploying test Shiny apps. Changed to normal library imports. Note, the chunk options are after library imports so packages are available.

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.

1 participant