Improvements to stall-monitoring service and script. #17127
Conversation
- Significant improvements to plotting efficiency. - Fixes to thread accounting--not fully solved, but better.
…s--update stall-monitoring script.
|
A new Pull Request was created by @knoepfel (Kyle Knoepfel) for CMSSW_9_0_X. It involves the following packages: FWCore/Concurrency @cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
|
please test |
|
The tests are being triggered in jenkins. |
|
Comparison job queued. |
cmsbuild
added
comparison-pending
tests-pending
and removed
comparison-available
tests-approved
labels
|
Pull request #17127 was updated. @cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please check and sign again. |
|
Most recent improvements include:
The accounting error I thought I had encountered before no longer seems to be present. I will adjust the script if it arises again. |
|
Please test |
|
The tests are being triggered in jenkins. |
cmsbuild
added
comparison-pending
tests-pending
and removed
comparison-available
tests-approved
labels
|
Pull request #17127 was updated. @cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please check and sign again. |
|
Assuming the tests pass, this PR is complete. |
|
please test |
|
The tests are being triggered in jenkins. |
|
Comparison job queued. |
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_9_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar |
cmsbuild
added
core-approved
fully-signed
and removed
core-pending
pending-signatures
labels
|
+1 |
Script now takes advantage of information emitted at the
preSourceEventandpostSourceEventsignals. In addition the script is significantly faster for creating the PDF image, and it also includes capabilities for creating a stack plot.