Skip to content

Add a Valve class #245

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

Draft
Jennefer Maldonado (jennmald) wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Add a Valve class #245

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
453560b
draft: value using two button shutter
jennmald Feb 27, 2026
5c03a81
move valve class
jennmald Mar 3, 2026
1b04422
better naming conventions
jennmald Mar 4, 2026
62cbe30
move stop functions
jennmald Mar 6, 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
5 changes: 4 additions & 1 deletion nslsii/devices.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@

_time_fmtstr = '%Y-%m-%d %H:%M:%S'


class TwoButtonShutter(Device):
# TODO: this needs to be fixed in EPICS as these names make no sense
# the value coming out of the PV does not match what is shown in CSS
Expand Down Expand Up @@ -128,3 +127,7 @@ def __init__(self, *args, **kwargs):
super().__init__(*args, **kwargs)
self._set_st = None
self.read_attrs = ['status']

class Valve(TwoButtonShutter):
Copy link
Copy Markdown
Contributor

@tacaswell Thomas A Caswell (tacaswell) Mar 4, 2026

Choose a reason for hiding this comment

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

At least at SRX we are (were?) using this for the a photon shutter, I think we need a better name.

Copy link
Copy Markdown
Contributor Author

@jennmald Jennefer Maldonado (jennmald) Mar 4, 2026

Choose a reason for hiding this comment

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

so something that captures, valves + photon shutters?

Copy link
Copy Markdown
Contributor

@tacaswell Thomas A Caswell (tacaswell) Mar 4, 2026

Choose a reason for hiding this comment

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

The interesting property is that "does not close on stop", putting more in a normal comment.

def stop(self, *, success=False):
Copy link
Copy Markdown
Contributor

@tacaswell Thomas A Caswell (tacaswell) Mar 4, 2026

Choose a reason for hiding this comment

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

This now does not close despite the name.

The default stop method is a no-op so we should move the stop from the other class to this one.

Copy link
Copy Markdown
Contributor Author

@jennmald Jennefer Maldonado (jennmald) Mar 6, 2026

Choose a reason for hiding this comment

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

I am not sure if I just made it worse.

...
Comment thread
tacaswell marked this conversation as resolved.
Outdated