-
Notifications
You must be signed in to change notification settings - Fork 9
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.
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
aps_undulator.py #973
Comments
I got started on this in Haven, FYI. I want energy, gap, etc to be positioners so I diverged from the apstools version, the other signals should be fine, though. Plan to test this on real hardware once our PV gateway gets updated. |
Need to make a list of the full set of undulators to be available. See the |
How different are the controls (and PVs) for these different insertion devices? Is there a common subclass? The S29ID-IEX insertion device control screen is not like the others. Perhaps consider the Planar as the base class, then look at 2M, 4M, STI, and Revolver as possible variations. Might learn that all can augment a common subclass. |
Of these designs, only the 4M is not associated with a feature beam line (or an early beam line such as 29ID). |
This is for the STI insertion device (#977)? |
NOTE: To get to these displays, log into an account and workstation on the oxygen network and run |
@prjemian I'm not sure actually. I wrote it against the Planar undulator, but I don't know if that's what we have. Is there a way to know which style of undulator we have at 25-ID? I sorted our the positioner bits. I can read the 4 positioner components, and every signal connects at 25-ID. I like the positioner approach because it makes the gap and tapers scannable. Since we can't move our ID yet, I haven't testing any of the set() features. |
The storage ring provides an ID_Main.adl screen which color-codes all the insertion devices. Here's a picture from the apstools issue: Show the color code of "Planar" for the ID at 25-ID. as the screen shows, that's the most common design. |
Makes sense. I also confirmed this with Marty Smith. I wrote the Device class assuming a planar undulator, so that's good. Anyway, let me know if you want it turned into a PR or anything. The positioner-style undulator device I wrote would NOT be backwards compatible with the apstools ApsUndulator Device. |
I recognize that the ApsUndulator is for the APS Undulator A. Not guaranteed to be a good base class or model for APS-U. It shares some basic ideas, but you (and we) want to make changes not. Primarily, energy and gap will be SoftPositioner subclasses. Most of the new undulators share this PV set. Others, notably, the Revolver, used at 8-ID and 34-ID, is different. |
A PR is most welcome, for issue #978. |
Pre-upgrade the undulator EPICS support did not include energy backlash motion. Does anybody know if this changed? We had to implement it in Bluesky before (link), which is not ideal. |
I'll ask the group (on Teams). |
Only the 29ID undulator issue #980 remains now and it is lower priority. Closing since all undulators are covered by issues. |
apstools/apstools/devices/aps_undulator.py
Line 20 in ece9663
apstools/apstools/devices/aps_undulator.py
Line 71 in ece9663
There are 6 types of ID:
The text was updated successfully, but these errors were encountered: