"Showstoppers" for CMSSW_9_4_0_pre3 (and thus CMSSW_9_4_0) closing

[davidlange6]

Please add your comments below.

• Need final understanding of DQM footprint for Tier0 (and rereco)

• [MERGED] OMTF ([94x] L1T OMTF introduce packer, unpacker, config DQM #20859)

• [MERGED] ~~ BMTF ([94x] L1T BMTF introduce packer and small emulator fix #20834)~~

• [MERGED] TwinMux ([94x] L1T TwinMux #20832)

• Pixel premixing problems still to be debugged (likely related to introduction of dynamic inefficiency in 2017 detector configuration)

• ECAL ZS/SR plan (may not affect software, but may have big impact on schedule (PPD Thursday, still uncertain plan)

• [MERGED] Pixel code protection Fix negative error cases #20904

• [MERGED] Tracking changes for inactive modules Introducing "Inactive" inner/outer hits... #20938

• muon sim information added to AOD

• [No change needed] HLT menu will be v3 (already integrated)

• [MERGED] ECAL digitization (comment below)

• [MERGED] L1 Calo ([94x] L1T Calo Layer1 packer unpacker #20914)

• [MERGED] high pt Egamma (Disabling high pt trained regression except for saturated E/gammas #20771)

• [RESOLVED] Resolution of HCAL geometry interface issues

• [MERGED] Pixel conditions update GT Updates with Pixel Local Reco conditions for V2 MC #20922

• [MERGED] Tau RECO change (increase of pT threshold for photons to be considered for tau reconstruction and ID #20900) - good to have?

[cmsbuild]

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]

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]

assign hlt

[cmsbuild]

New categories assigned: hlt

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

[Martin-Grunewald]

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):

• update the L1T menu to V4 in 94X and 92X GTs (request sent to AlCa)
• switch relvals to use V4 L1T + V4 HLT ( one-line change in Configuration/HLT/python/autoHLT.py)

[Martin-Grunewald]

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]

From reco:

More critical (affects products in RECO)

• Disabling high pt trained regression except for saturated E/gammas Disabling high pt trained regression except for saturated E/gammas #20771
  • it should converge today/tomorrow

Possibly critical (affects products in RECO)

• Fix negative error cases [in pixel local reco] Fix negative error cases #20904
  • "not a showstopper" according to Fix negative error cases #20904 (comment). Comparisons show some changes in data and MC.

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

• increase of pT threshold for photons to be considered for tau reconstruction and ID increase of pT threshold for photons to be considered for tau reconstruction and ID #20900
• Tensorflow-based integration of new DeepFlavour tagger Tensorflow-based integration of new DeepFlavour tagger #19893

[crovelli]

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]

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

[davidlange6]

#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]

Two PRs from AlCa/DB side needed for pre3 :

1. GT Updates with Pixel Local Reco conditions for V2 MC #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. add LumiCorrections records and definitions; adding LumiInfo.h members/methods #20651
   in pre-release so that further developments can be done as this will eventually become a PCL workflow.

3. [94X] more Strip conditions supported by Payload Inspector #20643
   this is for Payload inspector. Further developments are waiting for this to be merged.

@lpernie @ggovi

[arunhep]

assign alca

[cmsbuild]

New categories assigned: alca

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

[slava77]

New "showstopper" for reco category:

• Introducing "Inactive" inner/outer hits... (Introducing "Inactive" inner/outer hits... #20938)
  • Due to the "unexpected" increase in inactive ROCs we need a clear estimate of REAL tracking efficiency to answer questions such as "how many tracks do not have a BPIX1 hit?" w/o this PR such a question will be much harder to answer.

[arunhep]

+1

for 940_pre3