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

HQ Threat table path descriptions #1455

Closed
newtpatrol opened this issue Nov 20, 2023 · 0 comments · Fixed by #1456
Closed

HQ Threat table path descriptions #1455

newtpatrol opened this issue Nov 20, 2023 · 0 comments · Fixed by #1456
Assignees
Labels
in progress This issue is actively being worked on science request A request/proposal from within natcap related to science (rather than engineering) task Something needs to be done
Milestone

Comments

@newtpatrol
Copy link

The descriptions for the Threat table fields cur_path, base_path and fut_path don't actually say that they're file paths, and they should. Here's the current verbiage for cur_path:

cur_path (raster, required): Map of the threat’s distribution in the current scenario. Each pixel value is the relative intensity of the threat at that location.

I feel like this should say something like

File system path to a raster of the threat’s distribution in the current scenario. Each pixel value in the threat raster is the relative intensity of the threat at that location, with values between 0 and 1.

This is in investspec, so I can't change it. What I will change in the UG is making a larger Note regarding the relative nature of the paths (so that doesn't need to be replicated in each field's description, although it could be, added to the chapter text, not investspec).

@davemfish davemfish transferred this issue from natcap/invest.users-guide Nov 20, 2023
@davemfish davemfish added the task Something needs to be done label Nov 20, 2023
@davemfish davemfish added this to the 3.14.1 milestone Nov 20, 2023
@dcdenu4 dcdenu4 self-assigned this Nov 27, 2023
dcdenu4 added a commit to dcdenu4/invest that referenced this issue Nov 27, 2023
dcdenu4 added a commit to dcdenu4/invest that referenced this issue Nov 27, 2023
@dcdenu4 dcdenu4 added science request A request/proposal from within natcap related to science (rather than engineering) in progress This issue is actively being worked on labels Nov 27, 2023
dcdenu4 added a commit to dcdenu4/invest that referenced this issue Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress This issue is actively being worked on science request A request/proposal from within natcap related to science (rather than engineering) task Something needs to be done
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants