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

Phase2 duplicate pixels handling #39007

Open
jpata opened this issue Aug 9, 2022 · 9 comments
Open

Phase2 duplicate pixels handling #39007

jpata opened this issue Aug 9, 2022 · 9 comments

Comments

@jpata
Copy link
Contributor

jpata commented Aug 9, 2022

In #37559 (comment) it was noticed that duplicate pixels are not treated correctly in the Phase2 pixel digitizer. As I understand, their appearance is currently related to the simulation code.

In the longer term (as I suggested above) - certainly longer than the target of this PR (~ urgent for Run3 consistency ), I think the digitizer logic should be changed to avoid duplicate simDigis and just sum them up upstream of any recontruction.

In the meanwhile this can become an open cmssw issue with simulation signature

@jpata
Copy link
Contributor Author

jpata commented Aug 9, 2022

assign simulation

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 9, 2022

New categories assigned: simulation

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 9, 2022

A new Issue was created by @jpata Joosep Pata.

@Dr15Jones, @perrotta, @dpiparo, @rappoccio, @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 Aug 9, 2022

assign trk-dpg

@mmusich
Copy link
Contributor

mmusich commented Aug 9, 2022

@emiglior @suchandradutta FYI

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 9, 2022

New categories assigned: trk-dpg

@connorpa,@mmusich,@tsusa you have been requested to review this Pull request/Issue and eventually sign? Thanks

@mmusich
Copy link
Contributor

mmusich commented Sep 15, 2022

urgent

@srimanob
Copy link
Contributor

FYI @cms-sw/l1-l2
How do you use RECO track in L1T campaign? Could you please see the report above on degradation of tracks?

@srimanob
Copy link
Contributor

Hi @mmusich

Thanks for pinging me.
Should I assume the issue of tracks will propagate to objects, like electron, muon, jet right? Maybe we need to see how the issue will effect globally also.

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