CaltechOpticalObservatories / NGPS

NGPS Software
2 stars 0 forks source link

require TCS init before startup #87

Open astronomerdave opened 1 month ago

astronomerdave commented 1 month ago

Disable (grey-out) the STARTUP button until the TCS has been initialized.

image

Background: Originally, the STARTUP button in ngps-observe started all subsystems, including the TCS. Now that we've been required to remove the TCS-init from that automated procedure and into a separate manual control, the STARTUP button should be disabled (greyed-out) until the TCS has been initialized, because some subsystems require at least knowing which TCS is to be used (real / emulator)

astronomerdave commented 1 month ago

The sequencer now returns an error if STARTUP is sent before initializing the TCS, so maybe it's not necessary to grey out the STARTUP button. While it's no longer functionally necessary, perhaps it still should be greyed out to indicate that something else needs to be done first.