Add VeryLooseDNNnoCF97 electron WP
Adds new VeryLooseDNNnoCF97 WP and replaces LooseDNNnoCF in PHYSLITE as discussed in ATLASG-2789 . This MR targets the next PHYSLITE production so should be merged by 27/9.
Tagging @honguyen @mojeda @kristin @ludovica @ahabouel for e/y
As the variable in physlite is only replaced, the size will not change but still tagging @gwatts @ekourlit @emmat
Merge request reports
Activity
added Recommendation label
This merge request affects 4 packages:
- PhysicsAnalysis/Algorithms/EgammaAnalysisAlgorithms
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkEGamma
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys
- PhysicsAnalysis/ElectronPhotonID/ElectronPhotonSelectorTools
Affected files list will not be printed in this case
Adding @jcatmore ,@zmarshal ,@tadej ,@krumnack ,@dshope ,@emmat as watchers
- Resolved by Filip Nechansky
added analysis-interface-change label
CI Result FAILURE (hash e73cfd87) Athena AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info) This merge request affects 4 packages:
- PhysicsAnalysis/Algorithms/EgammaAnalysisAlgorithms
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkEGamma
- PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys
- PhysicsAnalysis/ElectronPhotonID/ElectronPhotonSelectorTools
Affected files list will not be printed in this case
Adding @tadej ,@krumnack ,@emmat ,@zmarshal ,@dshope ,@jcatmore as watchers
CI Result FAILURE (hash 1e0ad356) Athena AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info) removed analysis-review-required label
added analysis-review-approved label
added urgent label
- Resolved by Thomas Strebler
Hi @fnechans changes look good to me. Could you please resolve the open thread and the CI failure tests (see errors below) ? Thanks, L1 (Fábio).
ERROR Py:diff-root ERROR the following variables exist only in the old file ! ERROR Py:diff-root ERROR - [ElectronsAuxDyn.DFCommonElectronsDNNLooseNoCF] ERROR Py:diff-root ERROR the following variables exist only in the new file ! ERROR Py:diff-root ERROR - [ElectronsAuxDyn.DFCommonElectronsDNNVeryLooseNoCF97]
Edited by Fabio Lucio Lucio Alves
removed review-pending-level-1 label
added review-user-action-required label
added RC Attention Required label