Skip to content

SuperKEKB BPMs Re-Synchronisation script#572

Merged
fsoubelet merged 20 commits intomasterfrom
resync_bpms
Feb 18, 2026
Merged

SuperKEKB BPMs Re-Synchronisation script#572
fsoubelet merged 20 commits intomasterfrom
resync_bpms

Conversation

@fsoubelet
Copy link
Copy Markdown
Member

This is a port of pylhc/PyLHC#112 functionality to omc3.scripts. The initial PR has been done by @Mael-Le-Garrec and this one only includes a few tidying commits from me.

In pylhc/PyLHC#112 this has been checked and approved. I've let the very specific constants stay in the script file in this version. Doc is fine, tests are fine.

@fsoubelet fsoubelet requested review from JoschD and jgray-19 February 16, 2026 10:35
@fsoubelet fsoubelet requested a review from a team as a code owner February 16, 2026 10:35
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 16, 2026

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  omc3
  __init__.py
  omc3/scripts
  resync_bpms.py 217-220, 242
Project Total  

This report was generated by python-coverage-comment-action

Comment thread tests/unit/test_bpm_resync.py
Comment thread omc3/scripts/resync_bpms.py Outdated
Comment thread omc3/scripts/resync_bpms.py
Comment thread omc3/scripts/resync_bpms.py Outdated
Comment thread omc3/scripts/resync_bpms.py
Comment thread omc3/scripts/resync_bpms.py
Comment thread omc3/scripts/resync_bpms.py Outdated
Comment thread omc3/scripts/resync_bpms.py
Comment thread omc3/scripts/resync_bpms.py Outdated
Comment thread omc3/scripts/resync_bpms.py
Mael-Le-Garrec and others added 6 commits February 18, 2026 11:32
update doc

Co-authored-by: Joshua Gray <61458311+jgray-19@users.noreply.github.com>
Co-authored-by: Joshua Gray <61458311+jgray-19@users.noreply.github.com>
@fsoubelet fsoubelet merged commit 473c14a into master Feb 18, 2026
36 checks passed
@fsoubelet fsoubelet deleted the resync_bpms branch February 18, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants