Introduce a decision name for lines
Introduce a decision name for lines (which is always
name + "Decision") and used that to configure the
ANNSvc
, (implcitly the DecReports writer ExecutionReportsWriter
)
and the SelReportsMaker
.
Also add make_mc_track_info
maker for /Event/MC/TrackInfo
.
See #130 (closed) for motivation. Closes #130 (closed).
goes together with LHCb!2373 (merged) and Phys!662 (merged)
Merge request reports
Activity
added Persistency enhancement labels
added 10 commits
-
922a37dc...cd0b9fe0 - 8 commits from branch
master
- 399ca58b - Introduce a decision name for lines
- e807a5a5 - Add function to fetch MCTrackInfo
-
922a37dc...cd0b9fe0 - 8 commits from branch
mentioned in merge request Phys!662 (merged)
mentioned in merge request LHCb!2373 (merged)
added lhcb-gaudi-head label
- [2020-02-06 00:08] Validation started with lhcb-gaudi-head#2524
- [2020-02-06 00:22] Validation started with lhcb-gaudi-head#2524
- [2020-02-06 00:41] Validation started with lhcb-gaudi-head#2524
- [2020-02-07 00:12] Validation started with lhcb-gaudi-head#2525
- [2020-02-08 00:11] Validation started with lhcb-gaudi-head#2526
- [2020-02-09 00:08] Validation started with lhcb-gaudi-head#2527
- [2020-02-11 00:13] Validation started with lhcb-gaudi-head#2528
Edited by Software for LHCbmentioned in merge request LHCb!2319 (merged)
See failing tests in the nightlies that seem related: https://lhcb-nightlies.web.cern.ch/logs/tests/nightly/lhcb-gaudi-head/2525/x86_64-centos7-gcc9-opt/Moore/#failures_28
added 8 commits
-
e807a5a5...afee4577 - 2 commits from branch
master
- 93415b72 - Introduce a decision name for lines
- 1f3578a6 - Add function to fetch MCTrackInfo
- 922a37dc - Fixed formatting
- d6e38b80 - Merge branch 'master' into rmatev-add-decision
- 625fe60a - Fix sel report filtering
- e558800c - Merge branch 'rmatev-add-decision' of ssh://gitlab.cern.ch:7999/lhcb/Moore into rmatev-add-decision
Toggle commit list-
e807a5a5...afee4577 - 2 commits from branch
Locally I fixed the
hlt2_reco_baseline_multithreaded
test with !384 (merged). That just leaves the routing bits test, which also fails on master, so I think it is unrelated to this MR.- Resolved by Alex Pearce
We now also fixed the behaviour of the routing bits writing to rely on
line.decision_name
rather thanline.name
and now themdf_test_routing_bits
works locally.Edited by Ross John Hunter
mentioned in merge request !384 (merged)