Skip to content
Snippets Groups Projects

Make `rawbank_list` input configurable

Merged Giovanni Bassi requested to merge retinacluster_checks into master
All threads resolved!

Make the rawbank_list input of the make_transposed_raw_banks configurable.

This should be tested with LHCb!3456 (closed), Allen!803 (merged) and lhcb-datapkg/PRConfig!217 (merged).

Edited by Giovanni Bassi

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
  • Giovanni Bassi added 1 commit

    added 1 commit

    • 645920db - remove unused banks for hlt1_cosmics in calo tests

    Compare with previous version

  • Giovanni Bassi changed the description

    changed the description

    • Resolved by Rosen Matev

      I think this should be inferred from the configuration JSON file instead of configuring it explicitly, it's straightforward to parse the JSON file in python.

      P.S. Once HLT1 can be configured from TCK, the HltConfigSvs can be used to achieve the same thing in C++ when running from TCK.

  • mentioned in issue #388 (closed)

  • Giovanni Bassi added 1 commit

    added 1 commit

    • a7cf2464 - taking RawBank list from configuration file (JSON)

    Compare with previous version

  • Giovanni Bassi added 1 commit

    added 1 commit

    Compare with previous version

  • Giovanni Bassi added 2 commits

    added 2 commits

    • db681f42 - make list of RawBanks required configurable
    • ffda4a6c - taking RawBank list from configuration file (JSON)

    Compare with previous version

  • Rosen Matev resolved all threads

    resolved all threads

  • Rosen Matev approved this merge request

    approved this merge request

  • merged

  • Rosen Matev mentioned in commit 0ac11263

    mentioned in commit 0ac11263

  • mentioned in issue #411 (closed)

  • unassigned @msaur and @gbassi

  • added testing label

  • Please register or sign in to reply
    Loading