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

[Work in progress] Pcrafi post processor #5018

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from

Conversation

olivierdalang
Copy link
Contributor

WORK IN PROGRESS - DO NOT MERGE

What does it fix?

  • Ticket: #
  • Funded by: GA & SPC
  • Implemented by: Marco & Olivier
  • Description:
    • This adds a new postprocessor that assigns damage state to structures based on flood depths. It is though to work with buildings from the PCRAFI dataset used in the Pacific. It includes a vulnerability function (draft), a report element and a specific hazard classification.

Checklist:

  • Commit messages are descriptive and explain the rationale for changes
  • Add to the changelog in metadata.txt if it's a new feature
  • Unit test for new code added
  • Request someone to review or test your PR

@olivierdalang
Copy link
Contributor Author

As written in the description, the vulnerability function included is just a draft, so the PR isn't meant to be merged yet, but meant to show what we're up to here at SPC.

as hazard with value 0 is not aggregated, we get inconsistent
values in aggregation (DS0_count is 0 even if there are unaffected
buildings in that area, grand total shows correct count though)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants