Skip to content

Allow multiple denominators#105

Merged
kandrosov merged 4 commits intocms-flaf:mainfrom
kandrosov:v2603
Mar 23, 2026
Merged

Allow multiple denominators#105
kandrosov merged 4 commits intocms-flaf:mainfrom
kandrosov:v2603

Conversation

@kandrosov
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the normalisation-weight building in Corrections to support defining cross section and denominator branches from multiple processors simultaneously (distinguished by per-processor suffixes), enabling multiple denominator schemes in one run.

Changes:

  • Build a mapping of eligible processors for XS/denominator definition, including a required “default” processor and per-processor suffixing.
  • Update defineCrossSection / defineDenominator to define multiple branches and return the created branch names.
  • Update base weight construction to loop over all denominators (and corresponding XS branches) and emit weight_base{suffix}* outputs.

@kandrosov
Copy link
Copy Markdown
Contributor Author

tested with cms-flaf/FLAF#238

@kandrosov kandrosov merged commit a46b5cc into cms-flaf:main Mar 23, 2026
3 checks passed
@kandrosov kandrosov deleted the v2603 branch March 23, 2026 09:34
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