Skip to content

Extend logic of shared-cluster check in TrackListMerger #48004

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
missirol opened this issue May 3, 2025 · 8 comments
Open

Extend logic of shared-cluster check in TrackListMerger #48004

missirol opened this issue May 3, 2025 · 8 comments

Comments

@missirol
Copy link
Contributor

missirol commented May 3, 2025

CMSHLT-3534 describes a use case where track collections based on different SiStrip clusters would need to be combined taking into account the removal/cleaning of duplicate tracks.

As pointed out by @slava77, this does not work correctly out-of-the-box and, quoting him from CMSHLT-3534,

It should be relatively straightforward to extend the shared input/cluster check in case clusters are from different collections and in that case to explicitly check if the firstStrip+size ranges overlap for clusters from the same detId. It will need some care to keep it computationally efficient (so that this doesn't slow down the offline code significantly in particular).

In principle, this feature could be used to implement the changes described in CMSHLT-3534, and that would lead to an increase of ~2% in HLT throughput for the latest Run-3 HLT menu.

Is this something that tracking experts (e.g. @cms-sw/tracking-pog-l2) could implement ? (the sooner, the better)

@cmsbuild
Copy link
Contributor

cmsbuild commented May 3, 2025

cms-bot internal usage

@cmsbuild
Copy link
Contributor

cmsbuild commented May 3, 2025

A new Issue was created by @missirol.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@mmusich
Copy link
Contributor

mmusich commented May 5, 2025

assign reconstruction, hlt

@mmusich
Copy link
Contributor

mmusich commented May 5, 2025

type tracking

@cmsbuild
Copy link
Contributor

cmsbuild commented May 5, 2025

New categories assigned: reconstruction,hlt

@jfernan2,@mandrenguyen,@Martin-Grunewald,@mmusich you have been requested to review this Pull request/Issue and eventually sign? Thanks

@missirol
Copy link
Contributor Author

missirol commented May 6, 2025

@cms-sw/tracking-pog-l2

What do you think ? (useful / not useful, easy / hard, will be done / will not be done)

@mmasciov
Copy link
Contributor

mmasciov commented May 6, 2025

@cms-sw/tracking-pog-l2

What do you think ? (useful / not useful, easy / hard, will be done / will not be done)

@missirol, we are looking into it. Will comment here and/or with a PR ASAP.

@missirol
Copy link
Contributor Author

@mmasciov , any news on this front ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants