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
OMTF o2o upgrade #35876
OMTF o2o upgrade #35876
Conversation
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35876/26274
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
Hi @michalszleper please note that you have forgotten to do |
L1TriggerConfig/L1TConfigProducers/src/L1TMuonOverlapFwVersionOnlineProd.cc
Outdated
Show resolved
Hide resolved
L1TriggerConfig/L1TConfigProducers/src/L1TMuonOverlapFwVersionOnlineProd.cc
Show resolved
Hide resolved
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35876/26541
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35876/26597
|
Pull request #35876 was updated. @malbouis, @yuanchao, @cmsbuild, @rekovic, @ggovi, @tvami, @cecilecaillol, @francescobrivio can you please check and sign again. |
@cmsbuild , please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5e7108/20501/summary.html Comparison SummarySummary:
|
Ok it looks good to me, I dont have any further comments, @yuanchao anything from your side? |
Regardless to set GT via autoCond, I see most problems are from the deprecated ES:get and edm::EDAnalyzer are actually from the included L1T XXXviewers and XXXwriters. They need to be fixed in separated PRs. For others, I don't have further comments. It's ok with me. |
+alca |
+db |
+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:
Created a new OMTF configuration object L1TMuonOverlapFwVersion to be used in o2o, a dedicated ES producer and a new o2o online producer. Also necessary was an appropriate modification in L1TMuonOverlapObjectKeysOnlineProd.cc and little additions and modifications in several other packages in order to include the new record. There is also some standalone test code of the new record in Utilities, which is of lesser importance.
PR validation:
The code has been tested with o2o.sh independently by myself and Hyejin Kwon, with input from the OMDS and output in a local sqlite file. Everything was checked to work such as expected from o2o. Writing and reading the new record in a standalone way was also tested using the code in Utilities.
if this PR is a backport please specify the original PR and why you need to backport that PR: