cms-patatrack / cmssw

CMSSW fork of the Patatrack project
https://patatrack.web.cern.ch/patatrack/index.html
Apache License 2.0
2 stars 5 forks source link

Address HCAL review comments regarding CondFormats/HcalObjects #565

Closed mariadalfonso closed 3 years ago

mariadalfonso commented 4 years ago

as part of review of https://github.com/cms-sw/cmssw/pull/31720

abdoulline commented 4 years ago

Thank you, Maria! Also would like to explicitly add Igor @igv4321

fwyzard commented 3 years ago

Validation summary

Reference release CMSSW_11_2_0_pre8 at 3c9bd1ba78ad Development branch cms-patatrack/CMSSW_11_2_X_Patatrack at 6c2a63380102 Testing branch cms-patatrack/CMSSW_11_2_X_Patatrack at 6c2a63380102 with PRs:

Validation plots

/RelValTTbar_14TeV/CMSSW_11_2_0_pre7-PU_112X_mcRun3_2021_realistic_v8-v1/GEN-SIM-DIGI-RAW

/RelValZMM_14/CMSSW_11_2_0_pre7-112X_mcRun3_2021_realistic_v8-v2/GEN-SIM-DIGI-RAW

/RelValZEE_14/CMSSW_11_2_0_pre7-112X_mcRun3_2021_realistic_v8-v1/GEN-SIM-DIGI-RAW

Validation plots (CPU vs GPU)

/RelValTTbar_14TeV/CMSSW_11_2_0_pre7-PU_112X_mcRun3_2021_realistic_v8-v1/GEN-SIM-DIGI-RAW

/RelValZMM_14/CMSSW_11_2_0_pre7-112X_mcRun3_2021_realistic_v8-v2/GEN-SIM-DIGI-RAW

/RelValZEE_14/CMSSW_11_2_0_pre7-112X_mcRun3_2021_realistic_v8-v1/GEN-SIM-DIGI-RAW

Throughput plots

/EphemeralHLTPhysics1/Run2018D-v1/RAW run=323775 lumi=53

scan-136.885502.png zoom-136.885502.png scan-136.885512.png zoom-136.885512.png scan-136.885522.png zoom-136.885522.png

logs and nvprof/nvvp profiles

/RelValTTbar_14TeV/CMSSW_11_2_0_pre7-PU_112X_mcRun3_2021_realistic_v8-v1/GEN-SIM-DIGI-RAW

/RelValZMM_14/CMSSW_11_2_0_pre7-112X_mcRun3_2021_realistic_v8-v2/GEN-SIM-DIGI-RAW

/RelValZEE_14/CMSSW_11_2_0_pre7-112X_mcRun3_2021_realistic_v8-v1/GEN-SIM-DIGI-RAW

Logs

The full log is available at https://patatrack.web.cern.ch/patatrack/validation/pulls/68fd64ff1a572a5efce15eee5f4fcb46ea283c78/log .

fwyzard commented 3 years ago

No changes in the HCAL (or other) workflows, as expected.

fwyzard commented 3 years ago

I will do it for the Patatrack PR. Can you do it for the one for CMSSW ?

mariadalfonso commented 3 years ago

I will do it for the Patatrack PR. Can you do it for the one for CMSSW ?

uhm, the only PR I own (13/N) contains only the conditions for the CondFormats/HcalObjects, I didn't push there the files CUDADataFormats/HcalDigi

so both the patatrack and cms-sw (8/N) are yours

fwyzard commented 3 years ago

Ah, perfect, then I'll propagate the change myself with the next update.