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
MessageLogger parser script #29758
MessageLogger parser script #29758
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29758/15204
|
A new Pull Request was created by @fabiocos (Fabio Cossutti) for master. It involves the following packages: FWCore/MessageLogger @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
@fabiocos I suppose you have tested the script with python3? Could you give a short example of what it does? |
+1 |
Comparison job queued. |
@makortel yes, it compiles with python3, and in CMSSW_11_1_PY3_X_2020-05-06-2300 it behaves as expected. Just to make a simple example:
can be manipulated to extract with
an output like
You may select specified categories, level of verbosity, optionally print the message label, or the |
|
Comparison is ready Comparison Summary:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the example, looks useful indeed.
The code-checks are being triggered in jenkins. |
@makortel thank you for the suggestion, I have updated the code |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29758/15223
|
Pull request #29758 was updated. @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please check and sign again. |
please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 Thanks |
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. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR contains a refreshed version of an old python utility of mine that I have used for years to extract in a selective way MessageLogger information from log files. As I am recently sharing it with others to manipulate geometry validation dumps, I propose here in case it might be considered of more general interest.
FWCore/MessageLogger/scripts
looks a natural location to me, but of course it might be moved elsewhere in case.PR validation:
All options and combinations I have tested look to work as expected.