Skip to content
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

Jump to bottom

PDM: Make the UI to reflect changes on parameters (interlock, onoff, current_power) #17

Closed
mmouchous-ledger opened this issue Sep 26, 2024 · 2 comments · Fixed by #19
Closed

PDM: Make the UI to reflect changes on parameters (interlock, onoff, current_power) #17

mmouchous-ledger opened this issue Sep 26, 2024 · 2 comments · Fixed by #19
Assignees
@mmouchous-ledger

Comments

@mmouchous-ledger
Copy link
Member

mmouchous-ledger commented Sep 26, 2024 •
edited
Loading

Some parameters of the PDM change dynamically:

  • Sweep feature,
  • OnOff state / Interlock

The UI does not reflect those changes
@mmouchous-ledger mmouchous-ledger self-assigned this Sep 26, 2024
@mmouchous-ledger
Copy link
Member Author

mmouchous-ledger commented Nov 25, 2024 •
edited
Loading

Instruments have a pyqtSignal called parameter_changed. This feature should be used to trigger the update of the UI

@mmouchous-ledger
Copy link
Member Author

Also, it should add the possibility to get the Interlock status of the PDM and the On/Off state

@mmouchous-ledger mmouchous-ledger changed the title When the laser power is modified with the "Sweep" feature, it is not reflected in the UI PDM: Make the UI to reflect changes on parameters (interlock, onoff, current_power) Nov 25, 2024
@mmouchous-ledger mmouchous-ledger linked a pull request Nov 25, 2024 that will close this issue
Pdm UI autoupdate #19
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mmouchous-ledger mmouchous-ledger

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Pdm UI autoupdate Ledger-Donjon/laserstudio
1 participant
@mmouchous-ledger