prefer error in databroker when multiple algorithms declare the same output & remove some error messages
the error containing "does not produce any data" should not be at this place, should it? unless you want to "force" people to only put real producers into the m_producers rather than a full list of algorithms. And if so, it should not say "IGNORING...", since it actually sets algEntry->dependsOn
also removed the unused algorithm
warning since the counting is not yet implemented, and it annoys people currently ^^
@graven FYI
@olupton FYI put the error()
instead of warning()
for duplicated outputlocations
@sponce FYI: you might want to cherry-pick this and try whether this works out well..
Merge request reports
Activity
- [2018-11-10 00:03] Validation started with lhcb-lcg-dev3#707
- [2018-11-10 00:04] Validation started with lhcb-lcg-dev4#711
- [2018-11-10 00:06] Validation started with lhcb-tdr-test#356
- [2018-11-10 00:07] Validation started with lhcb-dd4hep#55
- [2018-11-10 00:15] Validation started with lhcb-gaudi-head#2056
- [2018-11-10 00:15] Validation started with lhcb-sanitizers#60
- [2018-11-11 00:09] Validation started with lhcb-sanitizers#61
- [2018-11-11 00:09] Validation started with lhcb-dd4hep#56
- [2018-11-11 00:09] Validation started with lhcb-gaudi-head#2057
- [2018-11-11 00:09] Validation started with lhcb-lcg-dev3#708
- [2018-11-11 00:10] Validation started with lhcb-tdr-test#357
- [2018-11-11 00:11] Validation started with lhcb-lcg-dev4#712
- [2018-11-12 00:04] Validation started with lhcb-lcg-dev4#713
- [2018-11-12 00:05] Validation started with lhcb-dd4hep#57
- [2018-11-12 00:06] Validation started with lhcb-lcg-dev3#709
- [2018-11-12 00:06] Validation started with lhcb-gaudi-head#2058
- [2018-11-12 00:09] Validation started with lhcb-sanitizers#62
- [2018-11-12 00:10] Validation started with lhcb-tdr-test#358
- [2018-11-12 08:55] Validation started with lhcb-gaudi-head#2058
- [2018-11-12 08:55] Validation started with lhcb-lcg-dev4#713
- [2018-11-12 08:55] Validation started with lhcb-lcg-dev3#709
- [2018-11-13 00:03] Validation started with lhcb-gaudi-head#2059
- [2018-11-13 00:03] Validation started with lhcb-sanitizers#63
- [2018-11-13 00:04] Validation started with lhcb-lcg-dev4#714
- [2018-11-13 00:04] Validation started with lhcb-lcg-dev3#710
- [2018-11-13 00:04] Validation started with lhcb-dd4hep#58
- [2018-11-13 00:07] Validation started with lhcb-tdr-test#359
- [2018-11-14 00:05] Validation started with lhcb-lcg-dev4#715
- [2018-11-14 00:08] Validation started with lhcb-dd4hep#59
- [2018-11-14 00:09] Validation started with lhcb-gaudi-head#2060
- [2018-11-14 00:11] Validation started with lhcb-sanitizers#64
- [2018-11-14 00:12] Validation started with lhcb-lcg-dev3#711
- [2018-11-14 00:17] Validation started with lhcb-tdr-test#360
Edited by Software for LHCbmentioned in merge request !801 (merged)
- Resolved by Niklas Stefan Nolte
- Resolved by Niklas Stefan Nolte
changed milestone to %v30r5
assigned to @leggett
@clemenci done
@nnolte : could not squash during the merge. do you want to
rebase -i
, or leave it as is?added 28 commits
-
e150f784...e1031fb8 - 27 commits from branch
gaudi:master
- 531802eb - removed some useless error messages
-
e150f784...e1031fb8 - 27 commits from branch
@leggett
rebase
d, waiting for the pipeline to go through