Skip to content

Changes to support hyperion-supervisor #1804

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rtuck99 merged 2 commits into from
Jan 15, 2026
Merged

Changes to support hyperion-supervisor #1804

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
52f7404
Add i03_supervisor beamline
rtuck99 Dec 9, 2025
46380d0
Add missing init file
rtuck99 Jan 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions src/dodal/beamlines/i03_supervisor.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
from dodal.common.beamlines.beamline_utils import set_beamline as set_utils_beamline

@jacob-williamson jacob-williamson Jan 9, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the purpose of having the supervisor as a beamline seperate to i03? Is this so you can dodal connect i03_supervisor to just get the baton?

@rtuck99 rtuck99 Jan 14, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The supervisor beamline is loaded into the bluesky context inside the hyperion-supervisor process. It only needs to access the baton as all the devices are interacted with by the hyperion-blueapi blueapi instance.

I think it's cleaner to do it this way rather than loading the main i03 beamline and disabling all the other devices through some special flag, loading i03.py is also quite heavyweight due to all the other things it imports.

@jacob-williamson jacob-williamson Jan 15, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense

from dodal.device_manager import DeviceManager
from dodal.devices.baton import Baton
from dodal.log import set_beamline as set_log_beamline
from dodal.utils import BeamlinePrefix, get_beamline_name

BL = get_beamline_name("i03")
set_log_beamline(BL)
set_utils_beamline(BL)

PREFIX = BeamlinePrefix(BL)


devices = DeviceManager()


@devices.factory()
def baton() -> Baton:
return Baton(f"{PREFIX.beamline_prefix}-CS-BATON-01:")
Empty file added src/dodal/testing/__init__.py
View file
Open in desktop
Empty file.