Skip to content
Snippets Groups Projects

Cleanup of muon trigger hypo package

Merged Savanna Shaw requested to merge sshaw/athena:muonHypoCleanup into master

Since the run 2 muon hypo code and the athenaMT hypo code is completely separate, I've moved all the athenaMT related code to a new package (TrigMuonHypoMT). This keeps the run 2 TrigMuonHypo from growing too large (there's all ready quite a lot there, not all of which will be needed for run 3), and will allow for easy cleanup of run 2 code later on when we don't need it any more.

We also had two different config files for running the athenaMT hypo code, so this was cleaned up to have one file (and the relevent tests were updated to be able to use only the one configuration). Also removed some unused/obsolete bits of the configuration.

cc @markowen @nakahama

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
  • Savanna Shaw added 1 commit

    added 1 commit

    Compare with previous version

  • This merge request affects 4 packages:

    • Trigger/TrigHypothesis/TrigMuonHypo
    • Trigger/TrigHypothesis/TrigMuonHypoMT
    • Trigger/TrigValidation/TrigUpgradeTest
    • Trigger/TriggerCommon/TriggerMenuMT
  • CI Result SUCCESS

    Athena AthSimulation
    externals
    cmake
    make
    required tests
    optional tests

    Full details available at NICOS MR-20816-2019-02-01-23-32
    Athena: number of compilation errors 0, warnings 1
    AthSimulation: number of compilation errors 0, warnings 0
    📝 CI Jenkins server is switched to https://atlas-sit-ci.cern.ch. It is accessible world-wide (behind CERN SSO). In old links to Jenkins server aibuild080.cern.ch:8080 should be replaced with atlas-sit-ci.cern.ch For experts only: Jenkins output [CI-MERGE-REQUEST 33166]

  • Frank Winklmeier resolved all discussions

    resolved all discussions

  • Tadej Novak resolved all discussions

    resolved all discussions

  • Tadej Novak resolved all discussions

    resolved all discussions

  • Looks OK now.

    Tadej (L1)

  • added review-approved label and removed review-pending-level-1 label

  • merged

  • Walter Lampl mentioned in commit 6084a1bf

    mentioned in commit 6084a1bf

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