-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
fix the cms EDModule type of GenParticleMatchMerger
#37613
Conversation
type bug-fix |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37613/29384
|
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages:
@cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fd5955/24002/summary.html Comparison SummarySummary:
|
hold
|
Pull request has been put on hold by @mmusich |
unhold |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37613/29394
|
Pull request #37613 was updated. @cmsbuild, @civanch, @mdhildreth can you please check and sign again. |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fd5955/24028/summary.html Comparison SummaryThere are some workflows for which there are errors in the baseline: Summary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
It appears that
CollectionAdder
cmssw/CommonTools/UtilAlgos/interface/CollectionAdder.h
Line 17 in 837fd56
which is the underlying module type of
cmssw/PhysicsTools/HepMCCandAlgos/plugins/GenParticleMatchMerger.cc
Line 4 in 6d2f660
is an
EDProducer
, but it was declared in the configuration as anEDFilter
:cmssw/SimTracker/TrackAssociation/python/allTrackMCMatch_cfi.py
Line 3 in 837fd56
leading to runtime errors of the type:
this is trivially fixed here.
In addition in commit f68a4df, I take care of some other mismatched types in the configuration.
The parameter
associator
ofMCTrackMatcher
should be acms.string
and not acms.InputTag
:cmssw/SimTracker/TrackAssociation/plugins/MCTrackMatcher.cc
Line 44 in 9583b99
Finally in the same commit, the value of the variable
associator
inTracker/TrackAssociation/python/trackMCMatch_cfi.py
is corrected such that it istrackAssociatorByHits
and notTrackAssociatorByHits
PR validation:
Private scripts.
if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A
cc: @tvami