search

slaclab / lcls-tools

Python tools for LCLS: post processing data, PV handling, pulling archive settings, etc.
Apache License 2.0
2 stars 21 forks source link

BDES not available for all magnets #115

Closed nneveu closed 11 months ago

nneveu commented 11 months ago

Describe the bug BDES is in the magnet object class, however we can not set BDES for all magnets. i.e. Bends are moved together through control of one magnet.

To Reproduce Steps to reproduce the behavior:

  1. Create magnet collection that includes bends.
  2. Try to set BDES for a magnet the does not control the bends.

Expected behavior We will not be able to set BDES for the non-control magnet.

Screenshots

Screenshot 2023-11-16 at 3 27 35 PM

Additional context This bug hasn't happened yet, but we know it will be an issue based on some magnets being stringed together.

nneveu commented 11 months ago

@MattKing06 's latest magnet updates should resolve this. He now checks which options are available.