Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • R Rec
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 289
    • Issues 289
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 61
    • Merge requests 61
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • Rec
  • Merge requests
  • !1040

Rich CK Resolution Fitter : Add new signal forms

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Christopher Rob Jones requested to merge RichCKResolutionFitter-UpdateFitShapes into run2-patches May 03, 2018
  • Overview 1
  • Commits 3
  • Pipelines 0
  • Changes 2

This MR updates the function used to fit the RICH CK resolution (used at the pit for the refractive index calibration). The aim is to give a slightly better fit to the data by

  • Allowing some asymmetry in the signal Normal, by way of allowing the +ve and -ve side resolution terms to be slightly different. Only small asymmetries are required, O(few %).
  • Use a hyperbolic background term instead of a cubic polynomial. Gives 'flatter' tails which seems to better fit, most of the time.

Fit has quite stringent checks on the convergence, and falls back to simpler forms if a fit fails, so should always provide a reliable result.

Following PDFs give examples of the old and new fits.

Run-175784-Old.pdf

Run-175784-New.pdf

I have submitted this against run2-patches, as I am more looking to run this in Run3. The current fit works fine as far as extracting the scale factor (the normal mean) and as it has been used for 2016, 2017 and 2018 (so far) I have decided to leave things as they are until the end of Run 2.

The fitter is not used as part of the HLT2/offline reconstruction, so should have no impact on any of the QM tests.

Please do propagate it to master though.

FYI @mmccann

Edited May 03, 2018 by Christopher Rob Jones
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: RichCKResolutionFitter-UpdateFitShapes