Skip to content
Snippets Groups Projects

prefer error in databroker when multiple algorithms declare the same output & remove some error messages

Merged Niklas Stefan Nolte requested to merge nnolte/Gaudi:NN_fixDataBroker into master

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..

Edited by Niklas Stefan Nolte

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Marco Clemencic changed milestone to %v30r5

    changed milestone to %v30r5

  • assigned to @leggett

  • @nnolte, can you have a look to the discussions?

  • added 1 commit

    Compare with previous version

  • Niklas Stefan Nolte resolved all discussions

    resolved all discussions

  • Niklas Stefan Nolte resolved all discussions

    resolved all discussions

  • Charles Leggett approved this merge request

    approved this merge request

  • @nnolte : could not squash during the merge. do you want to rebase -i, or leave it as is?

  • added 28 commits

    Compare with previous version

  • @leggett rebased, waiting for the pipeline to go through

  • Charles Leggett approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading