Skip to content

Conversation

@sawenzel
Copy link
Collaborator

  • do not apply a vertex when the mode is kNoVertex (this was buggy)

  • introduce a new vertex mode kCollContext to indicate the the vertex is to be taken from a collision context

* do not apply a vertex when the mode is kNoVertex
  (this was buggy)

* introduce a new vertex mode kCollContext to indicate
  the the vertex is to be taken from a collision context
@sawenzel sawenzel requested review from a team and jackal1-66 as code owners February 11, 2025 10:10
@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1

sawenzel added a commit to sawenzel/O2DPG that referenced this pull request Feb 11, 2025
This goes together with AliceO2Group/AliceO2#13955
and ensures a more consistent primary vertex location during
the event generator phases in O2DPG MC.

Fixes a bug where the primary vertex was applied twice.
@sawenzel sawenzel merged commit cf85a4c into AliceO2Group:dev Feb 11, 2025
15 checks passed
sawenzel added a commit to AliceO2Group/O2DPG that referenced this pull request Feb 11, 2025
This goes together with AliceO2Group/AliceO2#13955
and ensures a more consistent primary vertex location during
the event generator phases in O2DPG MC.

Fixes a bug where the primary vertex was applied twice.
@alcaliva alcaliva added the async-2024-pp-apass1 Request porting to async-2024-pp-apass1 label Feb 12, 2025
alcaliva pushed a commit to AliceO2Group/O2DPG that referenced this pull request Feb 12, 2025
This goes together with AliceO2Group/AliceO2#13955
and ensures a more consistent primary vertex location during
the event generator phases in O2DPG MC.

Fixes a bug where the primary vertex was applied twice.

(cherry picked from commit 7340626)
alcaliva pushed a commit to AliceO2Group/O2DPG that referenced this pull request Feb 12, 2025
This goes together with AliceO2Group/AliceO2#13955
and ensures a more consistent primary vertex location during
the event generator phases in O2DPG MC.

Fixes a bug where the primary vertex was applied twice.

(cherry picked from commit 7340626)
alcaliva pushed a commit to AliceO2Group/O2DPG that referenced this pull request Feb 12, 2025
This goes together with AliceO2Group/AliceO2#13955
and ensures a more consistent primary vertex location during
the event generator phases in O2DPG MC.

Fixes a bug where the primary vertex was applied twice.

(cherry picked from commit 7340626)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

async-2024-PbPb-apass1 async-2024-pp-apass1 Request porting to async-2024-pp-apass1

Development

Successfully merging this pull request may close these issues.

3 participants