-
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
avoiding trailing whitespaces in config dump #33634
avoiding trailing whitespaces in config dump #33634
Conversation
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33634/22500
|
A new Pull Request was created by @missirol (Marino Missiroli) for master. It involves the following packages: FWCore/ParameterSet @makortel, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Could you add a comment to this pull request showing an example of changes this makes to the output? That would be useful for documenting this change. |
Sure. Here are some selected examples (if a link to a full diff is more useful, it can be provided): process.DQMEventContent = cms.PSet(
outputCommands = cms.untracked.vstring(
- 'drop *',
+ 'drop *',
'keep *_MEtoEDMConverter_*_*'
),
splitLevel = cms.untracked.int32(0) RequireBothProjections = cms.bool(False),
ServiceParameters = cms.PSet(
Propagators = cms.untracked.vstring(
- 'SteppingHelixPropagatorAny',
- 'PropagatorWithMaterial',
+ 'SteppingHelixPropagatorAny',
+ 'PropagatorWithMaterial',
'PropagatorWithMaterialOpposite'
),
RPCLayers = cms.bool(True) cms.PSet(
default = cms.double(-1),
idx = cms.int32(1),
name = cms.string('leptonEtaRel_1'),
taggingVarName = cms.string('leptonEtaRel')
- ),
+ ),
cms.PSet(
default = cms.double(-1),
idx = cms.int32(0),
name = cms.string('leptonRatio_0'),
taggingVarName = cms.string('leptonRatio')
- ),
+ ),
cms.PSet(
default = cms.double(-1),
idx = cms.int32(1), constSiPar = cms.double(0.02),
deltasi_index_regemfac = cms.int32(3),
layerNoseWeights = cms.vdouble(
- 0.0, 39.500245, 39.756638, 39.756638, 39.756638,
+ 0.0, 39.500245, 39.756638, 39.756638, 39.756638,
39.756638, 66.020266, 92.283895, 92.283895
), |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9111ae/14884/summary.html Comparison SummarySummary:
|
+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. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
thanks Marino! |
+1 |
PR description:
This PR is an attempt to avoid trailing whitespaces in configuration file dumps. This was noticed by @fwyzard and @Sam-Harper (and maybe others), when integrating the config for the HLT Phase-2 menu in #32903.
PR validation:
Checked, with one configuration file, that the update gives the expected output (no trailing whitespaces). Example:
cmsDriver.py step3 --geometry Extended2026D49 --era Phase2C9 --conditions 111X_mcRun4_realistic_T15_v5 --processName RECO2 --step RAW2DIGI,RECO --eventcontent RECO --datatier RECO --filein /store/mc/Phase2HLTTDRSummer20ReRECOMiniAOD/QCD_Pt-15to3000_TuneCP5_Flat_14TeV-pythia8/FEVT/PU200_castor_111X_mcRun4_realistic_T15_v1-v1/100000/DA18C0FC-1189-D64B-B3B6-44F3F96F1840.root --mc --no_exec -n 10 --python_filename cfg.py --customise SLHCUpgradeSimulations/Configuration/aging.customise_aging_1000,Configuration/DataProcessing/Utils.addMonitoring edmConfigDump cfg.py > dump.py
If this PR is a backport, please specify the original PR and why you need to backport that PR:
N/A