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
Strip APV gains dumped in TTree #22129
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22129/3255 |
A new Pull Request was created by @jpriscia for master. It involves the following packages: CondTools/SiStrip @ggovi, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hi @jpriscia , thanks for this development - Just a suggestion : In the process of improving the documentation of our packages, it might be good to start to add a readme file explaining what is this APV gain reader and usage of the cfg - Thank you |
please test |
The tests are being triggered in jenkins. |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22129/3271 |
Hi @boudoul , readme added. Cheers |
Thanks, appreciated ! |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
the error reported in the addOn tests is not related to the code changes.
can tests be restarted? |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+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 now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
SiStripApvGainReader modified to dump gains in a TTree
Executable version of the cfg also added