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

Portfolio Assessment #88

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

Conversation

ioannis-vm
Copy link
Contributor

Portfolio Assessment

This PR is a first attempt to expand the capability of the command line approach to running an assessment, to be able to analyze multiple assets collectively in a single assessment while still being able to auto-populate configuration entries as done previously.

The commits contain a partial implementation, which needs to be reviewed and updated based on feedback before moving on, to minimize backtracking. Here is what is implemented so far:

  • Users run the pelicun command as before, but pass a specially formatted configuration file.
  • Pelicun recognizes it's a portfolio assessment and treats it as such.
  • The configuration file contains a list of asset IDs and corresponding AIM.json files, those that were typically used for a single assessment. It also identifies the response.csv file containing demands.
  • All individual AIM.json files should be auto-populated. This restriction confines the input space and reduces the need to validate the compatibility of all individual configuration files.
  • Pelicun auto-populates all AIM.json files, merges the component assignment files and runs the assessment.

We would need to discuss the format of the outputs.

Rename function and add note on side-effects.
Portfolio assessments won't have a `--demandFile` assignment, because
the individual demand files will be provided in the configuration file.
- Define test-case inputs. Will only include two buildings to keep the
  problem size manageable and ease validation.

- We'll use two HAZUS earthquake fragility curves. One should require
  auto-population, the other not, to cover both cases. Using available
  configuration files from the existing examples.
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