-
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
migration to DCSRecord
of GenericTriggerEventFlag
#37084
migration to DCSRecord
of GenericTriggerEventFlag
#37084
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37084/28566
|
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages:
@Martin-Grunewald, @rekovic, @epalencia, @emanueleusai, @ahmad3213, @cmsbuild, @missirol, @jfernan2, @pmandrik, @cecilecaillol, @pbo0, @rvenditti 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-fc14ca/22706/summary.html Comparison SummarySummary:
|
@mmusich many plots in Tracking folder that were previously filled, now get zero entries: |
268ae0f
to
df88baa
Compare
yes, I think the MC case was not handled correctly. Let's see if the last push resolved. |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37084/28580
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
df88baa
to
2046708
Compare
Many MEs (about 2300) are removed now and other 100s get fewer entries. I am not sure if this is due to last commits or to the interference of external commits |
#37090 this might play a large role here... |
Yes, indeed the changes from #37090 are the same. Thanks |
+1 |
Thanks for this update, @mmusich . As a side note to DQM, I noticed a DQM/TrackerCommon/interface/TriggerHelper.h, which looks somewhat similar to |
+hlt |
+l1 |
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 |
Thanks @missirol |
seems indeed unused in |
PR description:
In the same spirit of #29198 and #35782, in case the DCS status from SCAL is not available, fall back to
DCSRecord
data provided by the software FED n. 1022. The logic is modified in the classGenericTriggerEventFlag
which is used in several DQM clients.No differences are expected in Run1 and Run2 workflows. For Run3 data the selector should now work correctly.
PR validation:
Run successfully
runTheMatrix.py -l imited --ibeos
if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A