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 use of duplicate bin names in DQM/L1TMonitor #39853
Conversation
This is needed to avoid a ROOT error while merging during harvesting.
This avoids a ROOT error during harvesting.
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39853/32734
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @syuvivida, @pmandrik, @micsucmed, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test with workflow 4.6 |
please test workflow 4.6 |
urgent |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-542f14/28510/summary.html Comparison SummaryThere are some workflows for which there are errors in the baseline: @slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
@cms-sw/dqm-l2 could you please review this PR, and also sign if you agree with it? |
+1
|
merge |
+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. |
PR description:
ROOT appears to be enforcing that user defined bin names for histograms must be unique.
This fixes the failures in the integration builds of workflow 4.6.
PR validation:
Now when running workflow 4.6 step3 the jobs succeed.