Skip to content

Event-pools: Consistency fix for vertexing#2297

Open
sawenzel wants to merge 1 commit intomasterfrom
sawenzel-patch-2
Open

Event-pools: Consistency fix for vertexing#2297
sawenzel wants to merge 1 commit intomasterfrom
sawenzel-patch-2

Conversation

@sawenzel
Copy link
Contributor

For event pools, the vertex applied to events should (0,0,0). We tried to do this by setting 'kNoVertex' to the collision context... which unfortunately was not doing what we wanted. Instead of putting (0,0,0) this simply does not put a vertex into the collision context. This lead to the consequence that the vertex was fixed during event generation later.

Problem fixed by imposing kNoVertex also on the event generation step in case of event pool generation.

For event pools, the vertex applied to events should (0,0,0).
We tried to do this by setting 'kNoVertex' to the collision context... which unfortunately was not doing what we wanted. Instead of putting (0,0,0) this simply does not put a vertex into the collision context. This lead to the consequence that the vertex was fixed during event generation later.

Problem fixed by imposing `kNoVertex` also on the event generation step **in case of event pool generation**.
@sawenzel sawenzel requested a review from jackal1-66 as a code owner March 12, 2026 15:04
@github-actions
Copy link

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
async-2024-PbPb-apass2
async-2023-PbPb-apass5

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.

1 participant