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 new scenarios for HPBeamspot in ExpressAlignment HcalCalMinBias in HCalNZS PD #24573
Add new scenarios for HPBeamspot in ExpressAlignment HcalCalMinBias in HCalNZS PD #24573
Conversation
…nBias in HcaNZS PD
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-24573/6466 |
A new Pull Request was created by @tocheng (Tongguang) for master. It involves the following packages: Configuration/DataProcessing @cmsbuild, @franzoni, @fabiocos, @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. |
from Configuration.DataProcessing.Impl.hcalnzs import hcalnzs | ||
from Configuration.Eras.Era_Run2_2018_pp_on_AA_cff import Run2_2018_pp_on_AA | ||
|
||
class hcalnzsEra_Run2_2018_highBetaStar(hcalnzs): |
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.
the name is incorrect
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.
fixed in last commit
hcalnzs.__init__(self) | ||
self.recoSeq=':reconstruction_HcalNZS' | ||
self.cbSc='pp' | ||
self.addEI=True |
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.
apparently the 3 lines above are superfluous, they somehow sneaked in through copy-paste from old scenarios
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.
removed in last commit
#keep post-era parts the same as in the default 2018 era | ||
self.promptCustoms += [ 'Configuration/DataProcessing/RecoTLR.customisePostEra_Run2_2018' ] | ||
self.expressCustoms += [ 'Configuration/DataProcessing/RecoTLR.customisePostEra_Run2_2018' ] | ||
self.visCustoms += [ 'Configuration/DataProcessing/RecoTLR.customisePostEra_Run2_2018' ] |
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.
please use the same customs as in ppEra_Run2_2018_pp_on_AA.py
customisePostEra_Run2_2018_pp_on_AA
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.
changed to use customisePostEra_Run2_2018_pp_on_AA in the last commit
#keep post-era parts the same as in the default 2018 era | ||
self.promptCustoms += [ 'Configuration/DataProcessing/RecoTLR.customisePostEra_Run2_2018' ] | ||
self.expressCustoms += [ 'Configuration/DataProcessing/RecoTLR.customisePostEra_Run2_2018_express_trackingOnly' ] | ||
self.visCustoms += [ 'Configuration/DataProcessing/RecoTLR.customisePostEra_Run2_2018' ] |
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.
customs, as in hcalnzs, please use what's in ppEra
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.
changed to use customisePostEra_Run2_2018_pp_on_AA
please add the new scenarios to Configuration/DataProcessing/test/run_CfgTest.sh |
please abort |
Jenkins tests are aborted. |
… superfluous class initialization; add two scenarios to Configuration/DataProcessing/test/run_CfgTest.sh
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-24573/6467 |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-24573/6497 |
Pull request #24573 was updated. @cmsbuild, @franzoni, @fabiocos, @davidlange6 can you please check and sign again. |
please test |
The tests are being triggered in jenkins. |
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 be automatically merged. |
Hello all,
The PR introduces two new scenarios for Tier-0 to process HPBeamspot and HcalCalMinBias for heavyIon 2018 run in Run2_2018_pp_on_AA.
The new scenarios could be tested using the recent Heavy Ion test runs.