Skip to content

Conversation

@spjuhel
Copy link
Collaborator

@spjuhel spjuhel commented Dec 17, 2025

As the Risk trajectory PR is too substantial, this is a third split.

This PR implements the concept of impact computation strategies.
An impact computation strategy defines what underlying process to follow to get from (exp,haz,vul) triplet to an impact.

The idea is to ease the extension of the trajectories to also allow non ImpactCalc based impacts (think network, unsequa, etc.)

Currently, it really is just a wrapper around ImpactCalc().

It has proved helpful in tests/prototypes of risk trajectory / option appraisal to, for instance:

  • "custom" compute impacts on polygons / lines
  • pre-assign centroid or not

PR Author Checklist

PR Reviewer Checklist

@spjuhel
Copy link
Collaborator Author

spjuhel commented Dec 17, 2025

Although the class does not need Snapshot, the unit tests do, so they will fail until #1197 is merged.

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.

2 participants