Add zoomed dxy(PV) and dz(PV) to MultiTrackValidator, and a B-hadron TP instance to standalone/trackingOnly modes #17591
Conversation
In order to re-use the existing functionality, it was easiest to make HistoryBase::evaluate() public (they kind of were already, via TrackHistoryBase::evaluate()).
|
@cmsbuild, please test |
|
The tests are being triggered in jenkins. |
|
A new Pull Request was created by @makortel (Matti Kortelainen) for CMSSW_9_0_X. It involves the following packages: SimTracker/TrackHistory @civanch, @mdhildreth, @dmitrijus, @cmsbuild, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
|
+1 |
|
Comparison job queued. |
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_9_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar |
|
+1 |
This PR adds zoomed-in versions of dxy(PV) and dz(PV) to MultiTrackValidator, to better monitor the region of |dxy|<1cm that is important for b tagging. In addtion, an instance of MTV with B-hadron TrackingParticles is added to the standalone and trackingOnly sequences (possibly to be added to the standard sequence if proves to be useful in private tests).
Tested in 9_0_0_pre4, expecting appearance of
*_dxypv_zoomedand*_dzpv_zoomed*histograms. No changes are expected in existing histograms.@rovere @VinInn