-
Notifications
You must be signed in to change notification settings - Fork 20
Update ConsumedConditions scripts and add 2024 MC and DATA #101
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
base: master
Are you sure you want to change the base?
Update ConsumedConditions scripts and add 2024 MC and DATA #101
Conversation
|
Maybe I am missing something... but: cmsrel CMSSW_14_0_19
cd CMSSW_14_0_19/src/
cmsenv
git clone https://github.com/cms-AlCaDB/AlCaTools.git
git remote add francescobrivio https://github.com/francescobrivio/AlCaTools.git
git fetch francescobrivio
git cherry-pick dbe9182748eb574c2a00615ddc4ff875143ebe93
cd ConditionsConsumed/
./runCMSDrivers_mc2024.sh
head -70 output_step7_AODSIM.loggives: |
|
Ciao @mmusich indeed you are right, I hadn't noticed this crash hidden in the very long printout. The driver for step-7 is: which at (a very) first glance looks correct. The crash message seems to indicate a possible mismatch between CMSSW and the GT...I'll investigate further tomorrow. |
this error message is weird, because in principle that record label is used exclusively for the phase-0 tracking and indeed in so that label should not be looked at all (in normal conditions, and indeed I've never seen this failure in production mode) I can only speculate that |
|
I have tried to run the same step7-cmsDriver with:
|
|
Yes, indeed, I also came across with the same fatal exception, I discussed with @mmusich regarding that in MM, and since then I didn't find any other good solutions yet. When I ran first time, I also got the exception with Interestingly, we are using Yes, indeed, for some reason, it is trying to access a phase 0 pixel tracking record. |
PR Description
This PR updates the ConditionsConsumed sub-package:
--beamspotparameter to be--beamspot DBrealistic, as done in all CMS MC campaigns starting from 2024getAlCaCondInGT.pyConditionsConsumed/getAlCaCondInGT.pyto:checkTagInFilemethod to avoid clashing cases where part of the tag name is included in multiple Records.BeamSpotObjectsRcdandSimBeamSpotObjectsRcd, where the former was incorrectly marked as consumed in "GEN" step (while it's obviously not!).printAllRowmethod to avoid including in the output txt files 2 lines from the cmsDriver logs were the string/was present (Filter efficiency (taking into account weights)= (5) / (5) = 1.000e+00 +- 0.000e+00) which have nothing to do with the conditions: this was solved by looking in the log files for lines containing the stringfrontierinstead.PR Validation
The updated scripts were used to update the 2024 twiki: https://twiki.cern.ch/twiki/bin/view/CMS/AlCaDBHLT2024