Add more profile dump points to IgProfService #39071
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n_bugfix_12_4_X [12_4_X] Improved Egamma PFID model selection consistency
…_MC_production_12_4_X [12_4_X] Update L1T menu tag in MC and relval GTs
…WithDatasetPaths_124X update of `HLTriggerJSONMonitoring` for `DatasetPath`s [`12_4_X`]
[124X (backport)] Introduce era with CKF pixelLessStep: backport of cms-sw#38437
Prevents reading from uninitialised memory when the first event of a job has zero error words. Fixes a crash observed at HLT during data taking.
This reverts commit b62ed21.
…124X [12_4_X]Update HLT GT with ECAL DQM tower and channel status tags
…is no new valid measurement
…e no SiStrip Hits
- lowering pt - setting energy to 13.6 - renaming accordingly
[DT CSC RPC] Backport of bug fix to tag-and-probe in DQM (cms-sw#38028)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39071/31574
|
|
A new Pull Request was created by @makortel (Matti Kortelainen) for master. It involves the following packages:
@cmsbuild, @smuzaffar, @Dr15Jones, @makortel can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
…lock, and preEndJob signals
makortel
force-pushed
the
igprofservice
branch
from
98f015d
to
a0d1744
Compare
cmsbuild
added
alca-pending
analysis-pending
code-checks-pending
daq-pending
db-pending
dqm-pending
generators-pending
geometry-pending
heterogeneous-pending
hlt-pending
l1-pending
operations-pending
pdmv-pending
reconstruction-pending
simulation-pending
upgrade-pending
xpog-pending
and removed
code-checks-approved
labels
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39071/31577 ERROR: Unable to merge PR. See log https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39071/31577/cms-checkout-topic.log |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR description:
This PR adds options to
IgProfServiceto dump the profile atpreGlobalEndRun,{pre,post}EndProcessBlock, andpreEndJobsignals. I used those to investigate #38976.It turned out that the*EndProcessBlockfunctions were not run, for which I'll open a separate issue.PR validation:
Got profile dumps at the aforementioned transitions.