Place Moore RecSummary in a stream-dependent location

Tesla currently writes the RecSummary at /Event/Rec/Summary, but this is a problem for TurCal productions because the input files to Tesla already have a Rec/Summary from Brunel. This MR configures stream-specific RecSummary writers to place the summary in /Event/<stream>/Turbo/Rec/Summary.

The problem is that there is already logic for linking the summary from /Event to /Event/<stream>, but for this MR to be transparent to the user we might need a link going the other way.

/cc @rmatev @sstahl

LGTM. Not sure if we want the MR to be transparent in that way. In a way it already is because of the existing link + setting rootInTes.

The TupleToolRecoStats also checks the location with RootInTES and without. So users won't notice. Question is do we want this for the production or not?

resolved all discussions

resolved all discussions

added 1 commit

• f2ca8648 - Configure per-stream RecSummary decoder.

Compare with previous version

• [2017-07-18 14:36] Validation started with lhcb-gaudi-merge#171
• [2017-07-18 16:54] Validation started with lhcb-gaudi-merge#171
• [2017-07-19 00:04] Validation started with lhcb-old-stripping-tests#469
• [2017-07-19 00:09] Validation started with lhcb-gaudi-head-noavx#74
• [2017-07-19 00:10] Validation started with lhcb-lcg-dev3#224
• [2017-07-19 00:10] Validation started with lhcb-gaudi-head#1537
• [2017-07-19 00:10] Validation started with lhcb-head#1540
• [2017-07-19 00:10] Validation started with lhcb-clang-test#647
• [2017-07-19 00:10] Validation started with lhcb-gaudi-head-noavx2#55
• [2017-07-19 00:10] Validation started with lhcb-lcg-dev4#224
• [2017-07-19 00:27] Validation started with lhcb-2017-patches#83
• [2017-07-19 14:30] Validation started with lhcb-gaudi-merge#172

added 1 commit

• e550e8ec - Configure per-stream RecSummary decoder.

Compare with previous version

I don't know if the validation is triggered by hand, but if so: I just force-pushed to fix a configuration problem.

In this case yes, it was started by hand. Normally they start at midnight. Anyhow, this slot is not really relevant for this change.

I ran Tesla and DaVinci with this change and TupleToolRecoStats picks up the new location without any configuration. So now I'm happy with this

@erodrigu are you happy to merge this? We need to then cherry-pick it into v42r5-release

assigned to @erodrigu

Sure. DaVinci is actually failing to build but the issue is not related to this MR. Thanks for pinging me.

merged

mentioned in commit 21bcbae7

The (tesla functor cache) build is failing because the DEV version of TurboStreamProd is not up to date (/afs/cern.ch/lhcb/software/DEV/nightlies/DBASE/TurboStreamProd/v99r99).

@bcouturi @clemenci can we have the same for TurboStreamProd as we do for AppConfig?

mentioned in merge request !80 (merged)

mentioned in commit 8a27ce09