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
Improve the handling of module labels unsafe for DQM #38966
Conversation
Replace the unsafe characters in-place to avoid make unnecessary copies of the labels. Use the safe label for the name of the DQM histograms, but keep the original label for their titles.
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38966/31420
|
A new Pull Request was created by @fwyzard (Andrea Bocci) for master. It involves the following packages:
@Martin-Grunewald, @missirol can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
enable profiling |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0070ac/26643/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
Just for my education: where can one check the performance change of |
The pull request test results appears under the IB release they were tested against. For instance |
You click on the See Logs link next to "Profiling wf 11834.21 Results" and the select the json file like the one below. |
The files seems to be missing even though the logs indicate that the job ran without throwing any exceptions. |
It may take a file for the web server to register that the files are present. I can see them when I log directly into the web server host. |
Thanks for the info, @gartung ! |
+hlt |
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, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Follow up to #38963.
Replace the unsafe characters in-place to avoid make unnecessary copies of the labels.
Use the safe label for the name of the DQM histograms, but keep the original label for their titles.