Skip to content

Clean up the code which enables writing of /TRIGGER COOL folders into in-file metadata

This is a followup on the discussion at ATR-20272

This MR is a continuation of Vakho's !27104 (closed), after fixing conflicts and with the addition of a8133dc7.

CC: @tsulaia and @stelzer.

Changing if ('ds' in self.ConfigSrcList or self.writeESDAOD) to if ('ds' in self.ConfigSrcList and not self.writeESDAOD) removes the trigger COOL folders from the ESD and AOD formats. However - the xAOD::TriggerMenuAux in the AOD then is empty.

EDIT

I set the setupCOOLReading to be ESD creation only: if ('ds' in self.ConfigSrcList or self.writeESDAOD) and not self.writeAOD and TriggerFlags.configForStartup()!='HLToffline':

I set the setupxAODWriting execution of TrigConf__xAODMenuWriter to be run during ESD creation only (in ESDtoAOD the data get copied over).

This way we have non-empty collections also in the AOD 👍

Will un-WIP, but please comment @stelzer

Edited by Tim Martin

Merge request reports