Skip to content

[EMCAL-1037] Support for multiple classes per trigger type#2060

Merged
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
mfasDa:EMCAL-1037
Dec 4, 2023
Merged

[EMCAL-1037] Support for multiple classes per trigger type#2060
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
mfasDa:EMCAL-1037

Conversation

@mfasDa
Copy link
Copy Markdown
Collaborator

@mfasDa mfasDa commented Dec 1, 2023

  • Support for multiple classes per trigger type
  • Determine trigger cluster fromt the cluster in
    the CTP configuration instead of the class name
    (relevant for unanchored MC and 2022 data)
  • Require beam presence only when encoded in string

@mfasDa
Copy link
Copy Markdown
Collaborator Author

mfasDa commented Dec 1, 2023

Fixes 2 problems:
a) In PbPb there were several MB trigger classes, we want an and of them
b) Trigger class names had a different layout then the standard layout, for which parsing (adpated to 2023) failed. The new version is more general

@mfasDa
[EMCAL-1037] Support for multiple classes per trigger type
42cf26a 
- Support for multiple classes per trigger type
- Determine trigger cluster fromt the cluster in
  the CTP configuration instead of the class name
  (relevant for unanchored MC and 2022 data)
- Require beam presence only when encoded in string
@mfasDa
Copy link
Copy Markdown
Collaborator Author

mfasDa commented Dec 1, 2023

Tested on data and MC - ready to merge

@mfasDa
Copy link
Copy Markdown
Collaborator Author

mfasDa commented Dec 1, 2023

CI errors unrelated.

@mfasDa mfasDa mentioned this pull request Dec 1, 2023
Merged
@Barthelemy Barthelemy merged commit 04a43f3 into AliceO2Group:master Dec 4, 2023
Barthelemy pushed a commit that referenced this pull request Dec 4, 2023
@mfasDa @Barthelemy
[EMCAL-1037] Support for multiple classes per trigger type (#2060)
a665ada 
- Support for multiple classes per trigger type
- Determine trigger cluster fromt the cluster in
  the CTP configuration instead of the class name
  (relevant for unanchored MC and 2022 data)
- Require beam presence only when encoded in string
@mfasDa mfasDa deleted the EMCAL-1037 branch December 4, 2023 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Barthelemy Barthelemy Barthelemy approved these changes

@jokonig jokonig Awaiting requested review from jokonig jokonig is a code owner

@knopers8 knopers8 Awaiting requested review from knopers8 knopers8 is a code owner

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mfasDa @Barthelemy