Skip to content
Snippets Groups Projects

Fix calculations of one-strip cluster uncertainty

Closed Chav Chhiv Chau requested to merge chchau/athena:oneClusterError into master

Fixing a bug in the calculations of the uncertainty of one-strip clusters. sTGC one-strip clusters aren't usually associated to good tracks. So the bug doesn't have large effect, but it should be fixed.

Edit: The changes in the STGC_Measurements container is expected, since the uncertainties of one-strip clusters were wrong due to the bug.

Tagging @pscholer, @dpizzi, @alaurier

Edited by Chav Chhiv Chau

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

    • 46b4c0d8 - Update the calculation of the uncertainties

    Compare with previous version

  • This merge request affects 1 package:

    • MuonSpectrometer/MuonReconstruction/MuonDataPrep/STgcClusterization

    This merge request affects 1 file:

    • MuonSpectrometer/MuonReconstruction/MuonDataPrep/STgcClusterization/src/SimpleSTgcClusterBuilderTool.cxx

    Adding @apsallid ,@goetz ,@rosati ,@wleight ,@pscholer ,@stavrop ,@jojungge as watchers

  • added 1 commit

    Compare with previous version

  • This merge request affects 1 package:

    • MuonSpectrometer/MuonReconstruction/MuonDataPrep/STgcClusterization

    This merge request affects 1 file:

    • MuonSpectrometer/MuonReconstruction/MuonDataPrep/STgcClusterization/src/SimpleSTgcClusterBuilderTool.cxx

    Adding @apsallid ,@goetz ,@rosati ,@wleight ,@pscholer ,@stavrop ,@jojungge as watchers

  • :x: CI Result FAILURE (hash 46b4c0d8)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :o:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 64546]

  • :x: CI Result FAILURE (hash 48d94b06)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :o:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 64550]

  • Please confirm whether the CI test failure is expected and please resolve the outstanding thread.

    Jason (L1)

  • Chav Chhiv Chau changed the description

    changed the description

  • Chav Chhiv Chau resolved all threads

    resolved all threads

  • Author Contributor

    The CI test failure is expected. The fix of the error calculation changes the values in STGC_Measurements, and the changes are propagated to other containers. I edited the description to state the changes are expected. So I am removing the user-action-required label

    Edited by Chav Chhiv Chau
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading