-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
74x Fireworks: fix problem with muon global track propagation #11476
Conversation
A new Pull Request was created by @alja (Alja Mrak-Tadel) for CMSSW_7_4_X. Avoid merging globalTrack and Muon track reference merging. It involves the following packages: Fireworks/Muons @cmsbuild, @alja, @Dr15Jones can you please review it and eventually sign? Thanks. |
Undo change from the previous commit.
Pull request #11476 was updated. @cmsbuild, @alja, @Dr15Jones can you please check and sign again. |
fix for this particual case -- muon with global trackThe solution for this bug is that we don't add reference points from muon's globalTrack if there are already ordered 4 references points from existing inner and outer track position. Note: Filter muon.pt > 1 GeV does not help in this case because ordering of path marks is triggered at muon's track pt threshold > 1GeV which is not reached in this observed case even if muon.pt is more than 1GeV. |
sorting of path marks in fireworks::prepareTrackhttps://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Fireworks/Tracks/interface/TrackUtils.h#L54 This sorting is triggered only for pt > 1. Need to check how this algorithm covers case for negative charges and cases where the path mark is in the zone when direction of B field has changed. |
sorting of path marks in Tracker muonThis is the second case where path marks are merged from different sources. Here muon specific path marks come from matched muon chambers. The sorting of path marks depends on general track path marks sorting which need to be revisited. See comment above. |
Pull request #11476 was updated. @cmsbuild, @alja, @Dr15Jones can you please check and sign again. |
+1 |
The tests are being triggered in jenkins. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs after it passes the integration tests and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_7_6_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
+1 |
74x Fireworks: fix problem with muon global track propagation
Path marks from moun inner/outer track position and the last two path mark from muon->globalTrack():
innerTrack()->innerPosition() (-3.77266,-1.91869,5.69689)https://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Fireworks/Muons/src/FWMuonBuilder.cc#L266
moun innerTrack()->outerPosition() (-87.8516,-1.73888,263.885) https://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Fireworks/Muons/src/FWMuonBuilder.cc#L269
moun outerTrack()->innerPosition() (-112.689,29.4957,585.348)https://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Fireworks/Muons/src/FWMuonBuilder.cc#L275
moun outerTrack()->outerPosition() (-149.12,45.3741,839.739)https://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Fireworks/Muons/src/FWMuonBuilder.cc#L279
fireworks::prepareTrack track.innerOk (-3.77311,-1.9179,5.69697) https://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Fireworks/Tracks/interface/TrackUtils.h#L54
fireworks::prepareTrack track.outerOk (-129.255,35.7668,614.673)https://github.com/cms-sw/cmssw/blob/CMSSW_7_5_X/Fireworks/Tracks/interface/TrackUtils.h#L95
Yuu can see that there 4 reference path from muon inner and outer track position. Before that the globalMuon extra points are added, which resolves in the following path mark order:
TEveTrack 'TEveRecTrackT', number of path marks 6, label -1
Reference p: -0.696788 -0.354075 2.260245 Vertex: -3.773114e+00 -1.917895e+00 5.696975e+00 0 Extra:0.000000 0.000000 0.000000
Reference p: -0.058421 0.029470 0.185701 Vertex: -1.292555e+02 3.576676e+01 6.146726e+02 0 Extra:0.000000 0.000000 0.000000
Daughter p: 0.000000 0.000000 0.000000 Vertex: -3.772659e+00 -1.918688e+00 5.696890e+00 0 Extra:0.000000 0.000000 0.000000
Daughter p: 0.000000 0.000000 0.000000 Vertex: -8.785159e+01 -1.738879e+00 2.638853e+02 0 Extra:0.000000 0.000000 0.000000
Daughter p: 0.000000 0.000000 0.000000 Vertex: -1.126888e+02 2.949574e+01 5.853483e+02 0 Extra:0.000000 0.000000 0.000000
Decay p: 0.000000 0.000000 0.000000 Vertex: -1.491195e+02 4.537413e+01 8.397385e+02 0 Extra:0.000000 0.000000 0.000000