Skip to content

[v636][RF][HF] Check for consistent datasets across channels before combining#21139

Merged
guitargeek merged 1 commit intoroot-project:v6-36-00-patchesfrom
guitargeek:bp_2
Feb 3, 2026
Merged

[v636][RF][HF] Check for consistent datasets across channels before combining#21139
guitargeek merged 1 commit intoroot-project:v6-36-00-patchesfrom
guitargeek:bp_2

Conversation

@guitargeek
Copy link
Contributor

This makes problems like the one reported in #21066 easier to debug by the user, because there will be a meaningful error message like this instead of a crash:

[#2] FATAL:HistFactory -- ERROR: Inconsistent datasets across channel workspaces.
Workspace for channel "SR_3L_0RC_binary_binary_3L_0RC" does not match the datasets in channel "SR_2L_mva_2L_multiclass_ttll".
  Missing datasets:
    - obsData
All channel workspaces must contain exactly the same datasets.
terminate called after throwing an instance of 'RooStats::HistFactory::hf_exc'
  what():  HistFactory - Exception

(cherry picked from commit e37fefd)

This makes problems like the one reported in root-project#21066 easier to debug by
the user, because there will be a meaningful error message like this
instead of a crash:
```txt
[#2] FATAL:HistFactory -- ERROR: Inconsistent datasets across channel workspaces.
Workspace for channel "SR_3L_0RC_binary_binary_3L_0RC" does not match the datasets in channel "SR_2L_mva_2L_multiclass_ttll".
  Missing datasets:
    - obsData
All channel workspaces must contain exactly the same datasets.
terminate called after throwing an instance of 'RooStats::HistFactory::hf_exc'
  what():  HistFactory - Exception
```

(cherry picked from commit e37fefd)
@guitargeek guitargeek requested a review from lmoneta as a code owner February 3, 2026 21:49
@guitargeek guitargeek self-assigned this Feb 3, 2026
@guitargeek guitargeek merged commit 4852d6e into root-project:v6-36-00-patches Feb 3, 2026
20 of 21 checks passed
@guitargeek guitargeek deleted the bp_2 branch February 3, 2026 23:20
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.

1 participant