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
Add CSC and DT segments in miniAOD for highpt muons and enable muon match slimmer #19488
Conversation
A new Pull Request was created by @arizzi for master. It involves the following packages: PhysicsTools/PatAlgos @perrotta, @cmsbuild, @slava77, @monttj, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
In order to check that the slimming is safe, as suggested by Dima, I checked that pat::Muon::isSoftMuon(...) returns the same value (on 1k events thereare 2230 muons passing the isSoftMuon ID in both cases ) |
@cmsbuild please test workflow 136.7611,1325.5 |
The tests are being triggered in jenkins. |
@cmsbuild please abort |
Jenkins tests are aborted. |
@cmsbuild please test workflow 136.7611,1325.5 |
The tests are being triggered in jenkins. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1
|
@monttj can you please sign? |
std::vector<DTRecSegment4D> outDTSegmentsTmp; | ||
std::map<CSCSegmentRef,size_t> cscMap; | ||
std::vector<CSCSegment> outCSCSegmentsTmp; | ||
for(auto & seg : dtSegmentsRefs) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi @arizzi - this should be const &, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why? the container is a local set of refs ... why should it be const? (I mean, yes it could be const but is kind of irrelevant)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it seems purely cosmetic here.
@arizzi may I ask you to update this.
Thank you.
merge |
This PR adds DT and CSC segments in miniaod with relinking of the refs in muon matches to the new collection.
The current default is that segments are stored for muons with pt>50 (as the target is exo analyses at high pt).
In addition this PR enables minimal slimming of the muonmatches, this should be cross checked by the muon POG / experts to see if the IDs are stable against this change.
@gpetruc