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
Removal of the unused class of SiPixelCPEGenericDBErrorParametrization #35545
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35545/25774
|
A new Pull Request was created by @tvami (Tamas Vami) for master. It involves the following packages:
@malbouis, @yuanchao, @cmsbuild, @slava77, @jpata, @francescobrivio, @tvami can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild , please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20dea2/19420/summary.html Comparison SummarySummary:
|
Thanks, @tvami, for going ahead with the cleanup. What about the related CondFormat |
Hi @ferencek I removed some of what you are referring to. However there is an AlCa policy not to remove the record definition and registration. I'm not sure what to do with a record that has really never been used in production, we'll discuss this inside AlCa and I'll come back to this tomorrow |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35545/25780
|
@cmsbuild , please test |
Remove SiPixelFakeCPEGenericErrorParmESSource_cfi.py
e8154e4
to
8fe467e
Compare
Done |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-35545/25803
|
@cmsbuild , please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-20dea2/19449/summary.html Comparison SummarySummary:
|
Jenkins test pass nicely, no diff found, the SA only has a single warning for the bricked design |
+alca |
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 (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
I updated the PR description now @slava77 |
PR description:
Removal of the unused class of SiPixelCPEGenericDBErrorParametrization. It's been taken over by the PixelCPEGeneric algorithm years ago (about 2008). Please note that the corresponding tag from the GTs have been removed in the past already, this is just the code cleaning part of it.
Edit:
CondTools/SiPixel
PR validation:
Tested with
if this PR is a backport please specify the original PR and why you need to backport that PR:
Not a backport and no backport is needed
resolves cms-AlCaDB/AlCaTools#33