Skip to content
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

review and possibly disable looper reconstruction for HeavyIons pp_on_* #35758

Closed
slava77 opened this issue Oct 20, 2021 · 10 comments
Closed

review and possibly disable looper reconstruction for HeavyIons pp_on_* #35758

slava77 opened this issue Oct 20, 2021 · 10 comments

Comments

@slava77
Copy link
Contributor

slava77 commented Oct 20, 2021

This is a follow up to #35721 (comment)

in that PR the looper reconstruction was disabled for pp starting from Run-2.

To reduce divergence in the configuration and perhaps even improve the HI tracking, including getting somewhat faster tracking,
the proposal for the software is to drop removal of trackingNoLoopers in the HI pp_on_* eras.

@abaty @mandrenguyen

@cms-sw/tracking-pog-l2

@slava77
Copy link
Contributor Author

slava77 commented Oct 20, 2021

assign reconstruction

@cmsbuild
Copy link
Contributor

New categories assigned: reconstruction

@slava77,@jpata you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

A new Issue was created by @slava77 Slava Krutelyov.

@Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@mmusich
Copy link
Contributor

mmusich commented May 6, 2022

Partially addressed at #38288

@mmusich
Copy link
Contributor

mmusich commented Jun 9, 2022

type tracking

@jpata
Copy link
Contributor

jpata commented Jun 10, 2022

It was discussed here: #38288 that looper reco will be disabled for Run2_2018 and Run3.

In the future, it might be good to investigate other periods of Run2 for a consistent setup.

@mandrenguyen
Copy link
Contributor

+reconstruction
Loopers were disabled in pp_on_* as the PR title says.
I don't think it makes sense to revisit earlier ion running periods than 2018

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

@mandrenguyen
Copy link
Contributor

Closed #35758 as completed.

@mmusich
Copy link
Contributor

mmusich commented Nov 29, 2022

please close

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

5 participants