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.

Sign up for GitHub

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

Jump to bottom

Add new greenspace balance rasters to Urban Nature Access #1177

Closed
phargogh opened this issue Feb 8, 2023 · 2 comments · Fixed by #1178
Closed

Add new greenspace balance rasters to Urban Nature Access #1177

phargogh opened this issue Feb 8, 2023 · 2 comments · Fixed by #1178
Assignees
@phargogh
Labels
enhancement New feature or request in progress This issue is actively being worked on
Milestone
3.13.0

Comments

@phargogh
Copy link
Member

phargogh commented Feb 8, 2023 •
edited
Loading

@lmandle and @Yingjie4Science are asking for 2 additional files to be added to the outputs directory and are described here: https://docs.google.com/document/d/1uWJaJpbSzCX8XfCsDn44Euaad3ikYtjlctU2AlMk5Xk/edit

The files themselves should be trivial to implement; documentation will be harder.

I plan on doing these as a separate PR from the open #1150 .

@lmandle mentioned that for now we should only to these for total population, not per population group. The per-population group rasters can be added later on if there's demand. That should be a separate issue if so.

Time estimate: 1-3 hours to PR submission.
@phargogh phargogh added the enhancement New feature or request label Feb 8, 2023
@phargogh phargogh added this to the 3.13.0 milestone Feb 8, 2023
@phargogh phargogh self-assigned this Feb 8, 2023
@phargogh phargogh added the in progress This issue is actively being worked on label Feb 8, 2023
phargogh added a commit to natcap/invest.users-guide that referenced this issue Feb 9, 2023
@phargogh
Renaming greenspace_balance to greenspace_balance_percapita.
401ac87 
RE:natcap/invest#1177
phargogh added a commit to natcap/invest.users-guide that referenced this issue Feb 9, 2023
@phargogh
Documenting the new greenspace demand output.
76dac5e 
RE:natcap/invest#1177
phargogh added a commit to natcap/invest.users-guide that referenced this issue Feb 9, 2023
@phargogh
Reworking greenspace balance for the total population.
6b5fffa 
RE:natcap/invest#1177
phargogh added a commit to phargogh/invest that referenced this issue Feb 9, 2023
@phargogh
Renaming greenspace_balance -> greenspace_balance_percapita.
097f9a5 
RE:natcap#1177
phargogh added a commit to phargogh/invest that referenced this issue Feb 9, 2023
@phargogh
Implementing greenspace demand.
92a5c5a 
RE:natcap#1177
phargogh added a commit to phargogh/invest that referenced this issue Feb 9, 2023
@phargogh
Implementing greenspace_supply_totalpop as requested.
3cd5d4e 
This was already in the model, just under the confusing name 'greenspace
supply/demand budget.'.

RE:natcap#1177
@phargogh phargogh mentioned this issue Feb 9, 2023
Merged
@phargogh
Copy link
Member Author

phargogh commented Feb 9, 2023

It ended up making sense to include some of the per-popgroup rasters as part of this PR, so those are rolled in to #1178.

@phargogh
Copy link
Member Author

Resolved in #1178

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@phargogh phargogh

Labels
enhancement New feature or request in progress This issue is actively being worked on
Projects
None yet
Milestone
3.13.0
Development

Successfully merging a pull request may close this issue.

Urban Nature Access: Update greenspace balance rasters phargogh/invest
1 participant
@phargogh