Skip to content
Snippets Groups Projects

Tests for reco sequence with muon matching

Merged Claire Prouve requested to merge cprouve_muonmatchingtest into master

Tests setting up the hlt1 reco sequence with muon matching

  • only the .ref.x86_64-opt files are valid, the others have to be updated (- the mc-checking part contains a gentle hack because "MuonMatch" tracks are not valid for the PrChecker yet)

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
  • Alex Pearce
  • assigned to @apearce

  • Alex Pearce added 1 deleted label

    added 1 deleted label

  • assigned to @cprouve

  • Claire Prouve added 1 commit

    added 1 commit

    • 2f6e29b5 - replacing hack by using same principle as for standalone_hlt2_reco + applying...

    Compare with previous version

  • Claire Prouve added 1 commit

    added 1 commit

    Compare with previous version

  • unassigned @cprouve

  • Alex Pearce resolved all threads

    resolved all threads

  • Edited by Software for LHCb
  • I think all the non-opt tests have now completed, so you should be able to grab the references now @cprouve.

    The differences look pretty large to me though, e.g. in the dbg build (looking at the "CountersMismatch" field):

        (SciFiTrackForwarding ref) Search S1UV | 53
        (SciFiTrackForwarding new) Search S1UV | 180
        (SciFiTrackForwarding ref) Created long tracks | 935 | 47 | 0.050267
        (SciFiTrackForwarding new) Created long tracks | 935 | 138 | 0.14759
        (SciFiTrackForwarding ref) Search S2 UV | 76
        (SciFiTrackForwarding new) Search S2 UV | 248
        (SciFiTrackForwarding ref) Search S3UV | 411
        (SciFiTrackForwarding new) Search S3UV | 1337
        (SciFiTrackForwarding ref) Accepted input tracks | 71
        (SciFiTrackForwarding new) Accepted input tracks | 240
        (SciFiTrackForwarding ref) Search S1 | 93
        (SciFiTrackForwarding new) Search S1 | 293
        (SciFiTrackForwarding ref) Search S2 | 251
        (SciFiTrackForwarding new) Search S2 | 835
        (SciFiTrackForwarding ref) Search S3 | 2166
        (SciFiTrackForwarding new) Search S3 | 7546

    Are such big differences expected?

    • Author Developer
      Resolved by Alex Pearce

      When I moved the velo_track_min_ip parameter into the standalone_hlt1_muonmatching_reco method I changed it's default value. I assume the difference is from there. Sorry.

  • Alex Pearce resolved all threads

    resolved all threads

  • Claire Prouve added 1 commit

    added 1 commit

    Compare with previous version

  • Alex Pearce added all-slots label and removed 1 deleted label

    added all-slots label and removed 1 deleted label

  • Claire Prouve added 1 commit

    added 1 commit

    Compare with previous version

  • Alex Pearce mentioned in merge request !285 (merged)

    mentioned in merge request !285 (merged)

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