search

cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.07k stars 4.28k forks source link

"Showstoppers" for CMSSW_9_4_0_pre3 (and thus CMSSW_9_4_0) closing #20867

Closed davidlange6 closed 5 years ago

davidlange6 commented 6 years ago

Please add your comments below.

cmsbuild commented 6 years ago

A new Issue was created by @davidlange6 David Lange.

@davidlange6, @Dr15Jones, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

Martin-Grunewald commented 6 years ago

We (TSG) would like L1T v4 and HLT v4 to be used for the v2 MC production. The v4 menus require a set of new GTs (as the L1T menu is part of the GT) and a HLT PR with the HLT v4 menu dump.

Martin-Grunewald commented 6 years ago

assign hlt

cmsbuild commented 6 years ago

New categories assigned: hlt

@Martin-Grunewald,@silviodonato,@fwyzard you have been requested to review this Pull request/Issue and eventually sign? Thanks

Martin-Grunewald commented 6 years ago

To use V4 (L1T/HLT): The HLT V4.0 PRs are available as #20872 (94X) and #20873 (92X). They are mergable on their own as they do not yet switch relvals to HLT v4.

Still to do (needs to be done together in a PR):

Martin-Grunewald commented 6 years ago

A problem has been found online with the L1T menu v4, both in its original incarnation and in its slimmed version (the latter asked for inclusion into the GTs) - it leads to crashes of the HLT.

Given that the problem is not understood and it is unclear whether it is the firmware or also affecting the xml file (the latter is used for the GTs), TSG coordination decided to stay with the current V3 setup for the big MC production, rather than moving to V4.

Hence the deployment of the GTs with L1T v4 should be delayed until the problem is understood, at which time we either use those updated GTs or need to request new ones due to a fix in the xml file. This affects both 92X and 94X.

I am sorry that this issue requires a change of the originally envisioned plan.

(We still ask for integration of the HLT PRs ADDING the HLT v4, it is just a delay into the switchover of GTs and @relval2017).

slava77 commented 6 years ago

From reco:

More critical (affects products in RECO)

Possibly critical (affects products in RECO)

Less critical (affects only miniAOD or can be redone from AOD)

crovelli commented 6 years ago

Dear all, in ECAL we are checking the value of one parameter entering the sim to digi step, https://github.com/cms-sw/cmssw/blob/master/SimCalorimetry/EcalSimProducers/python/ecalNotContainmentSim_cff.py It may have slightly changed due to the recent changes in the simulated geometry. Things should converge by Friday, but this is to alert you. Thanks and sorry for the very short notice - ECAL DPG

Martin-Grunewald commented 6 years ago

TSG request for pre3: #20872 (94X).

davidlange6 commented 6 years ago

20872 is not a showstopper as it won't be used for MC prod. (but of course if it gets integrated that will be great)

arunhep commented 6 years ago

Two PRs from AlCa/DB side needed for pre3 :

  1. https://github.com/cms-sw/cmssw/pull/20922 : Important for V2 MC production. This contains Pixel local reco conditions for V2 MC. Tracker alignment group will use pre3 relvals to produce misalignment scenario.

  2. https://github.com/cms-sw/cmssw/pull/20651 in pre-release so that further developments can be done as this will eventually become a PCL workflow.

  3. https://github.com/cms-sw/cmssw/pull/20643 this is for Payload inspector. Further developments are waiting for this to be merged.

@lpernie @ggovi

arunhep commented 6 years ago

assign alca

cmsbuild commented 6 years ago

New categories assigned: alca

@cerminar,@arunhep,@franzoni,@lpernie,@ghellwig you have been requested to review this Pull request/Issue and eventually sign? Thanks

slava77 commented 6 years ago

New "showstopper" for reco category:

arunhep commented 6 years ago

+1

for 940_pre3