You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's some information that's hard-coded into the BTPS screens, and is duplicated by information in btps.py.
Expected Behavior
Information is not duplicated.
Context / environment
The same (or similar) information is already build into btps.py . When updating the stage mapping I fixed it in the code but forgot to do it in the screen, which caused confusion. I want to prevent this in the future.
Find a way to work the information present in btps.py into the screens. btps.py uses PVs, the screens use happi.... one method should be used, not both.
The text was updated successfully, but these errors were encountered:
Current Behavior
There's some information that's hard-coded into the BTPS screens, and is duplicated by information in
btps.py
.Expected Behavior
Information is not duplicated.
Context / environment
The same (or similar) information is already build into
btps.py
. When updating the stage mapping I fixed it in the code but forgot to do it in the screen, which caused confusion. I want to prevent this in the future.btps.py
:pcdsdevices/pcdsdevices/lasers/btps.py
Lines 661 to 714 in 665c1f4
Screens:
Related issue: #1269
Suggested Solution
Find a way to work the information present in
btps.py
into the screens.btps.py
uses PVs, the screens usehappi
.... one method should be used, not both.The text was updated successfully, but these errors were encountered: