Skip to content

feat: MultiComponentBoundTrackParameters::merge#5137

Open
andiwand wants to merge 19 commits intoacts-project:mainfrom
andiwand:feat-multitrackparams-toSingle
Open

feat: MultiComponentBoundTrackParameters::merge#5137
andiwand wants to merge 19 commits intoacts-project:mainfrom
andiwand:feat-multitrackparams-toSingle

Conversation

@andiwand
Copy link
Contributor

@andiwand andiwand commented Feb 20, 2026

Adds ability to convert MultiComponentBoundTrackParameters to BoundTrackParameters by utilizing central component merging.

--- END COMMIT MESSAGE ---

blocked by

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

📊: Physics performance monitoring for 49f0551

Full contents

physmon summary

@andiwand andiwand marked this pull request as ready for review February 20, 2026 19:30
@andiwand andiwand marked this pull request as draft February 21, 2026 17:44
@andiwand andiwand changed the title feat: MultiComponentBoundTrackParameters::toSingleComponent feat: MultiComponentBoundTrackParameters::merge Feb 23, 2026
@andiwand andiwand added the 🛑 blocked This item is blocked by another item label Feb 23, 2026
kodiakhq bot pushed a commit that referenced this pull request Feb 25, 2026
Currently we can only merge gaussian mixtures if we have a covariance. But extracting the mixed parameters does not require a covariance. This change extracts mixture functions for the parameters and reuses them if also a covariance is provided.

Useful for #5137

Apart from this
- doc is cleaned out and improved
@andiwand andiwand removed the 🛑 blocked This item is blocked by another item label Feb 25, 2026
@andiwand andiwand marked this pull request as ready for review February 25, 2026 23:18
@sonarqubecloud
Copy link

JustusRudolph pushed a commit to JustusRudolph/acts that referenced this pull request Feb 26, 2026
Currently we can only merge gaussian mixtures if we have a covariance. But extracting the mixed parameters does not require a covariance. This change extracts mixture functions for the parameters and reuses them if also a covariance is provided.

Useful for acts-project#5137

Apart from this
- doc is cleaned out and improved
Copy link
Member

@benjaminhuth benjaminhuth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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