-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
ECAL : management of PF RecHits Thresholds and Laser Alphas parameters #26522
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26522/9395
|
A new Pull Request was created by @depasse for master. It involves the following packages: CalibCalorimetry/EcalTrivialCondModules @christopheralanwest, @tocheng, @cmsbuild, @franzoni, @tlampen, @ggovi, @pohsun can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
The tests are being triggered in jenkins. |
-1 Tested at: 7fd7410 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: ClangBuild
I found compilation warning while trying to compile with clang. Command used:
See details on the summary page. |
Comparison not run due to Build errors/Fireworks only changes/No short matrix requested (RelVals and Igprof tests were also skipped) |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26522/9659
|
@depasse thank you, much better now. @tocheng @christopheralanwest @ggovi As far as I am concerned this PR is ready to go after the routine test |
unhold |
please test with cms-sw/cmsdist#4934, cms-data/CalibCalorimetry-EcalTrivialCondModules#5 |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@tocheng @christopheralanwest @pohsun @ggovi the changes after your signatures are just moving data files from CMSSW (and this PR) to the external data package, see cms-data/CalibCalorimetry-EcalTrivialCondModules#5 Unless you have comments by tomorrow, I will move forward with merging both this package and the corresponding external |
+1 |
merge @tocheng @christopheralanwest @ggovi as anticipated I move forward with the integration, please sign it for future reference |
+1 |
PR description:
To obtain sqlite_files of Particle Flow RecHits thresholds with different conditions and from txt files (easy to configure). To obtain sqlite files from experimental measurements of Laser Alphas (in txt format).
PR validation:
Successful cmsRun Tests with python files and txt files EB_thresholds_-1.txt, EE_thresholds_-1.txt, LaserLossRing.txt :
CondTools/Ecal/python/copyEcalPFRecHitThresholdsFromFile_cfg.py
CondTools/Ecal/test/alpha/create_alpha_trivial_byLaser_v6.py
if this PR is a backport please specify the original PR:
Before submitting your pull requests, make sure you followed this checklist: