Skip to content

Use the LHC orbit duration for CTP scaler rates - #15831

Merged
sawenzel merged 1 commit into
AliceO2Group:devfrom
sawenzel:sawenzel/ctp-scaler-orbit-duration
Sep 23, 2026
Merged

sawenzel merged 1 commit into
AliceO2Group:devfrom
sawenzel:sawenzel/ctp-scaler-orbit-duration

Conversation

@sawenzel

Copy link
Copy Markdown
Collaborator

This fixes the orbit-to-time conversion in the CTP scaler rates.

  • Rates used 88 us per orbit instead of 88.92 us, so they were 1.05 % too high.
  • For run 569757 (2 colliding bunches) this put the TVX rate above the colliding-bunch rate.
  • The conversion now uses o2::constants::lhc::LHCOrbitMUS, as getRatesForIndex already does.

https://its.cern.ch/jira/browse/O2-7064

This fixes the orbit-to-time conversion in the CTP scaler rates.

- Rates used 88 us per orbit instead of 88.92 us, so they were 1.05 % too high.
- For run 569757 (2 colliding bunches) this put the TVX rate above the colliding-bunch rate.
- The conversion now uses o2::constants::lhc::LHCOrbitMUS, as getRatesForIndex already does.

https://its.cern.ch/jira/browse/O2-7064

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sawenzel

Copy link
Copy Markdown
Collaborator Author

This renders the IR of run physical:

┌──────────────────┬─────────┬──────────────┬───────────────────────────┬──────────────┐
  │       run        │ library │ mid-run rate │ intervals above the limit │ p at mid-run │
  ├──────────────────┼─────────┼──────────────┼───────────────────────────┼──────────────┤
  │ 569757           │ release │ 22587.2 Hz   │ 174 of 174                │ 1.00428      │
  ├──────────────────┼─────────┼──────────────┼───────────────────────────┼──────────────┤
  │ 569757           │ with this PR │ 22352.3 Hz   │ 0 of 174                  │ 0.99384      │
  ├──────────────────┼─────────┼──────────────┼───────────────────────────┼──────────────┤
  │ 571043 (control) │ release │ 501132.1 Hz  │ 0                         │ 0.02130      │
  ├──────────────────┼─────────┼──────────────┼───────────────────────────┼──────────────┤
  │ 571043 (control) │ with this PR │ 495921.6 Hz  │ 0                         │ 0.02108      │
  └──────────────────┴─────────┴──────────────┴───────────────────────────┴──────────────┘

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 2598071 at 2026-09-23 00:02:

No log files found

Full log here.

@sawenzel
sawenzel merged commit b1e73be into AliceO2Group:dev Sep 23, 2026
11 checks passed
@sawenzel
sawenzel deleted the sawenzel/ctp-scaler-orbit-duration branch September 23, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants