Skip to content

hwp_supervisor: Add options to disable shutdown#1074

Open
ykyohei wants to merge 5 commits into
mainfrom
hwp_disable_shutdown
Open

hwp_supervisor: Add options to disable shutdown#1074
ykyohei wants to merge 5 commits into
mainfrom
hwp_disable_shutdown

Conversation

@ykyohei

@ykyohei ykyohei commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Description

Added a option to disable shutdown on startup
Added a task to disable and re-enable shutdown non-temporarily.

Motivation and Context

Having these option is helpful for hwp expert to operate in irregular state.
For example, sometimes lakeshore or ups could have communication issue and we cannot monitor YBCO temperature or ups state, but hwp could be safe to operate.

How Has This Been Tested?

Tested at satp3 on June 9th 2026.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

temporarily disable spin check

fix docs

debug

revert hwp gripper
@ykyohei ykyohei requested a review from bbixler500 June 10, 2026 01:14

@mhasself mhasself left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Definitely a useful thing. Can you please add (if not already in there):

  • the "shutdown_enabled" value should appear in session.data.
  • the update_shutdown task logs what value the user passed in (I don't think we'll see it in the logs, otherwise).

Also, is there a feed where this can be published? I vaguely remember that HWPSupervisor doesn't record much about the decisions it's making; consider starting this.

@ykyohei

ykyohei commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

@mhasself Thanks for the suggestions, updated as suggested and tested in daq-dev.
We recently updated hwp_supervisor to publish shutdown state so I also added "shutdown_enabled" as a new feed to publish.
Made similar updates to another PR of hwp-gripper as well #1075

@ykyohei ykyohei requested a review from mhasself June 10, 2026 15:06
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