Skip to content
Snippets Groups Projects

Remove DAQKernel package (and migrate its users, eg. `bankKiller` and `RawEventDump`)

Merged Gerhard Raven requested to merge remove-decoder-base-stuff into master

Retire Decoder::ToolBase and Decoder::AlgBase which were helpers to search for RawBanks from a search path of RawEvents.

  • migrate bankKiller and RawEventDump from Decoder::AlgBase to Gaudi::Functional::MergingConsumer
  • migrate IMuonRawBuffer and MuonRawBuffer to Rec into the sole component source directory which needs them
  • remove dictionary entry for IMuonRawBuffer (as that interface should be deprecated -- see #338 (moved))
  • move the search code for RawBank which used to be in Decoder::CommonBase to the one bit of code remaining which used it, namely ODINDecodeTool (which itself may be obsolete/deprecated)
  • remove the entire DAQKernel package, as its only raison d'etre was to host Decoder::ToolBase and Decoder::AlgBase.

must be used in conjunction with Rec!3698 (merged) which adds IMuonRawBuffer and MuonRawBuffer to Rec, and Allen!1386 (merged) which removes mention of DAQKernel from its dependencies (Allen didn't actually depend on DAQKernel). Same for MooreOnline!308 (merged). The change in printout in bankKiller requires Boole!531 (merged) which updates the references specifically for that change.

Edited by Gerhard Raven

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
  • Gerhard Raven added 1 commit

    added 1 commit

    • 483ecfd4 - Cleanup DecoderBase and its usage

    Compare with previous version

  • Gerhard Raven resolved all threads

    resolved all threads

  • mentioned in issue #338 (moved)

  • Gerhard Raven added 13 commits

    added 13 commits

    Compare with previous version

  • Edited by Software for LHCb
  • Gerhard Raven marked this merge request as ready

    marked this merge request as ready

  • Gerhard Raven changed the description

    changed the description

  • Gerhard Raven added 1 commit

    added 1 commit

    • 9ba48f74 - Cleanup DAQKernel and code using it

    Compare with previous version

  • Gerhard Raven marked this merge request as draft

    marked this merge request as draft

  • Gerhard Raven changed the description

    changed the description

  • Gerhard Raven added 1 commit

    added 1 commit

    • 7e625c90 - Cleanup DAQKernel and code using it

    Compare with previous version

  • Gerhard Raven added 1 commit

    added 1 commit

    • cc00589c - Cleanup DAQKernel and code using it

    Compare with previous version

  • Sebastien Ponce resolved all threads

    resolved all threads

  • Gerhard Raven changed the description

    changed the description

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