Skip to content

fix: Spacepoint-Track association in GNN algorithm#5263

Merged
kodiakhq[bot] merged 9 commits into
acts-project:mainfrom
benjaminhuth:refactor/update-odd-module-map-script-2
Mar 21, 2026
Merged

fix: Spacepoint-Track association in GNN algorithm#5263
kodiakhq[bot] merged 9 commits into
acts-project:mainfrom
benjaminhuth:refactor/update-odd-module-map-script-2

Conversation

@benjaminhuth

@benjaminhuth benjaminhuth commented Mar 19, 2026

Copy link
Copy Markdown
Member

Fixes the association of spacepoints to tracks that was broken since #5088.

Also improve python scripts and add track finder performance writer and track selection to have meaningful metrics.

--- END COMMIT MESSAGE ---

Any further description goes here, @-mentions are ok here!

  • Use a conventional commits prefix: quick summary
    • We mostly use feat, fix, refactor, docs, chore and build types.
  • A milestone will be assigned by one of the maintainers

@benjaminhuth benjaminhuth requested a review from andiwand March 19, 2026 16:34
@github-actions github-actions Bot added this to the next milestone Mar 19, 2026
@github-actions github-actions Bot added Component - Examples Affects the Examples module Changes Performance labels Mar 19, 2026
andiwand
andiwand previously approved these changes Mar 19, 2026
andiwand
andiwand previously approved these changes Mar 19, 2026
@github-actions

github-actions Bot commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

📊: Physics performance monitoring for a43edc6

Full contents

physmon summary

@benjaminhuth benjaminhuth requested a review from andiwand March 20, 2026 12:13
@benjaminhuth benjaminhuth changed the title refactor: Improve GNN ODD module map test fix: Spacepoint-Track association in GNN algorithm Mar 20, 2026
andiwand
andiwand previously approved these changes Mar 20, 2026
@sonarqubecloud

Copy link
Copy Markdown

@kodiakhq kodiakhq Bot merged commit 7f1ca5f into acts-project:main Mar 21, 2026
42 checks passed
benjaminhuth added a commit that referenced this pull request Mar 22, 2026
Requires a new module map file due to changes in the definition of the
geometric cuts. For now, the new patched module map files have the
min/max values of the affected `deta` and `diff_dydx` cuts inflated to
`+/- inf` to avoid regeneration of the module map files.

Should go in after #5263 

--- END COMMIT MESSAGE ---

Any further description goes here, @-mentions are ok here!

- Use a *conventional commits* prefix: [quick
summary](https://www.conventionalcommits.org/en/v1.0.0/#summary)
- We mostly use `feat`, `fix`, `refactor`, `docs`, `chore` and `build`
types.
- A milestone will be assigned by one of the maintainers

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@andiwand andiwand modified the milestones: next, v47.0.0, v46.0.0 Mar 23, 2026
benjaminhuth added a commit to benjaminhuth/acts that referenced this pull request Mar 23, 2026
Fixes the association of spacepoints to tracks that was broken since acts-project#5088.

Also improve python scripts and add track finder performance writer and track selection to have meaningful metrics.
benjaminhuth added a commit to benjaminhuth/acts that referenced this pull request Mar 23, 2026
Requires a new module map file due to changes in the definition of the
geometric cuts. For now, the new patched module map files have the
min/max values of the affected `deta` and `diff_dydx` cuts inflated to
`+/- inf` to avoid regeneration of the module map files.

Should go in after acts-project#5263 

--- END COMMIT MESSAGE ---

Any further description goes here, @-mentions are ok here!

- Use a *conventional commits* prefix: [quick
summary](https://www.conventionalcommits.org/en/v1.0.0/#summary)
- We mostly use `feat`, `fix`, `refactor`, `docs`, `chore` and `build`
types.
- A milestone will be assigned by one of the maintainers

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

2 participants