Skip to content

fix: interlock and bl state improvements#930

Open
d-perl wants to merge 6 commits into
mainfrom
fix/interlock_react_to_removed_state
Open

fix: interlock and bl state improvements#930
d-perl wants to merge 6 commits into
mainfrom
fix/interlock_react_to_removed_state

Conversation

@d-perl
Copy link
Copy Markdown
Contributor

@d-perl d-perl commented Jun 2, 2026

  • interlock actor and manager react to changes in available beamline states
  • don't excessively call unlock when not needed
  • on server startup, wait for beamline states to load before the actor evaluates

@d-perl d-perl force-pushed the fix/interlock_react_to_removed_state branch from d9763b0 to 129a28f Compare June 2, 2026 14:42
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 2, 2026

Codecov Report

❌ Patch coverage is 86.84211% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
bec_server/bec_server/actors/scan_interlock.py 40.00% 0 Missing and 3 partials ⚠️
bec_lib/bec_lib/bl_state_manager.py 80.00% 1 Missing ⚠️
..._server/bec_server/actors/builtin_actor_manager.py 94.11% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@d-perl d-perl force-pushed the fix/interlock_react_to_removed_state branch from 66d74c9 to 5dcb756 Compare June 3, 2026 09:03
@d-perl d-perl marked this pull request as ready for review June 3, 2026 09:19
@d-perl d-perl requested review from a team and wakonig June 3, 2026 09:19
@d-perl d-perl changed the title fix: interlock and bl state imrpovements fix: interlock and bl state improvements Jun 3, 2026
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.

2 participants