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
Update eventcontent structure for RecoHiEgamma #29746
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29746/15175
|
A new Pull Request was created by @jeongeun (JeongEun Lee) for master. It involves the following packages: RecoHI/HiEgammaAlgos @perrotta, @cmsbuild, @slava77 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 |
Comparison job queued. |
@mandrenguyen FYI |
'keep EESrFlagsSorted_ecalDigis__*', | ||
'keep recoPFCandidates_particleFlowEGamma_*_*', | ||
'keep recoPFCandidates_particleFlowTmp_*_*', | ||
"drop recoPFClusters_*_*_*", |
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.
This looks useless, are you as not keeping it anywhere here
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.
Thanks a lot! Okay, I'll remove the drop (L64).
And I'd like to notice to you that there are more things to be fixed.
I've realized that the floatedmValueMap_hi*
(L7-17) is not in the AOD event content, is in the RECO/FEVT event content.
So I'll fix it and also remove some unused modules (not found in the event content list from HI workflows output) in RecoHiEgammaRECO.
Comparison is ready Comparison Summary:
|
Status: waiting for an update of the PR. |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29746/15329
|
please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison 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. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Update event content definitions to explicitly have RECO to be a superset of AOD and for FEVT to be a superset of RECO. 1 file changed :
RecoHI/HiEgammaAlgos/python/RecoHiEgamma_EventContent_cff.py
(The previous tasks for RecoHI event content is PR#29576 )
PR validation:
Event Content comparison check was also done and there is no change with these updates.
Tested in CMSSW_11_1_X, the basic test all passed in the CMSSW PR instructions.