-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[12X][change GT] 2018 realistic #23498
Conversation
please test |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23498/5054 |
The tests are being triggered in jenkins. |
A new Pull Request was created by @lpernie (Luca Pernie) for master. It involves the following packages: Configuration/AlCa @arunhep, @cerminar, @cmsbuild, @franzoni, @pohsun, @lpernie can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
-1 Tested at: 934468a You can see the results of the tests here: I found follow errors while testing this PR Failed tests: RelVals
When I ran the RelVals I found an error in the following worklfows: runTheMatrix-results/10824.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+HARVESTFull_2018+ALCAFull_2018/step5_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+HARVESTFull_2018+ALCAFull_2018.log |
Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped) |
Error, need to be fixed by DQM: Begin processing the 1st record. Run 1, Event 1, LumiSection 1 on stream 0 at 06-Jun-2018 00:46:58.767 CEST |
@schneiml @dmitrijus can you please help us in solving the crash message observed in one of the DQM modules here : https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23498/28502/runTheMatrix-results/10824.0_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+HARVESTFull_2018+ALCAFull_2018/step5_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2018_GenSimFull+DigiFull_2018+RecoFull_2018+HARVESTFull_2018+ALCAFull_2018.log @fabiocos this is a limiting issue now for the delivery of Strips G2 calibration for 10.2.X MC because they need the MC sample with the similar GT provided in this PR and are facing the crash in DQM step. FYI @mmusich : did you manage to contact Tk-DQM ? |
@arunhep, I mentioned the issue to @fioriNTU. --- a/DQMOffline/CalibTracker/python/ALCARECOSiStripCalMinBiasDQM_cff.py
+++ b/DQMOffline/CalibTracker/python/ALCARECOSiStripCalMinBiasDQM_cff.py
@@ -13,6 +13,7 @@ ALCARECOSiStripCalMinBiasTrackingDQM = DQM.TrackingMonitor.TrackingMonitor_cfi.T
AlgoName = 'ALCARECO'+__selectionName,
FolderName = "AlCaReco/"+__selectionName,
BSFolderName = "AlCaReco/"+__selectionName+"/BeamSpot",
+ MeasurementState = "default",
# margins and settings
TkSizeBin = 71,
TkSizeMin = -0.5, by the way this |
Fix made here #23523 |
please test with #23523 |
The tests are being triggered in jenkins. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
superseeded by #23543 |
Fix for AlCaSiStripCalMinBias as discussed on PR #23498
…_1_X [10.1.X] Fix for ALCARECOSiStripCalMinBiasTrackingDQM as discussed on PR #23498
SUMMARY
One GT changed: 102X_upgrade2018_realistic_v2
. https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/102X_upgrade2018_realistic_v1/102X_upgrade2018_realistic_v2
Note
. This change is needed for the sample production to derive the G2 calibration for MC in 10.2.X
. Due to a broken paths in the DQM code this PR will fail
. This need to be fixed, because a new release with the updated AlCaReco Trigger bit is needed in order to generate the samples to derive the G2 calibration for MC in 10.2.X