Skip to content

Models for sensitivity analyses - #76

Merged
ZoeMZou merged 5 commits into
mainfrom
models-for-sensitivity-analyses
Oct 7, 2026
Merged

ZoeMZou merged 5 commits into
mainfrom
models-for-sensitivity-analyses

Conversation

@ZoeMZou

@ZoeMZou ZoeMZou commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

@ZoeMZou
ZoeMZou requested review from marwaalarab and a balanced review from Copilot October 6, 2026 16:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The sensitivity pipeline is consistently wired through the registry, dependencies, model inputs, and output paths; only a minor comment correction remains.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Extends the modelling pipeline to run main and sensitivity analyses consistently.

Changes:

  • Adds sensitivity variants to the analysis registry.
  • Routes models to sensitivity-specific cleaned datasets and dependencies.
  • Writes merged sensitivity outputs to dedicated subdirectories.
File Description
analysis/​active_analyses/​active_analyses.R Registers uniquely named sensitivity analyses.
analysis/​model/​fn-prepare_model_input.R Selects the appropriate cleaned dataset.
analysis/​create_project_actions.R Generates sensitivity model and output actions.
analysis/​make_output/​make_model_output.R Writes merged outputs to variant-specific directories.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

}

# Optional fourth argument selects the merged output folder.
# YAML needs selects the model outputs available to this merge action.

@marwaalarab marwaalarab left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@ZoeMZou
ZoeMZou merged commit 1eeb8ff into main Oct 7, 2026
@ZoeMZou
ZoeMZou deleted the models-for-sensitivity-analyses branch October 7, 2026 15:20
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.

3 participants