updates sequencerd published messages for fineacquire#409
Open
astronomerdave wants to merge 8 commits intomainfrom
Open
updates sequencerd published messages for fineacquire#409astronomerdave wants to merge 8 commits intomainfrom
astronomerdave wants to merge 8 commits intomainfrom
Conversation
adds FALIED state adds seqmon utility (as proxy for GUI)
* fixes bug in camerad emulator * updates seqmon utility
Contributor
Author
|
Sequencer PUB-SUB visibility migration note: The intended UI model: Changes needed in pygui/: @prkrtg
Meanwhile, I've added a utility
|
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




These are the updates needed to the sequencerd publishing to support fine acquire. Several legacy UDP async broadcasts have been removed to the GUI will need some changes:
ngps_gui.py— initialize_services()Add subscriptions for "seq_seqstate" and "seq_broadcast" alongside the existing "seq_waitstate" subscription
Connect a new sequencer_ready_signal (see below) to self.show_popup("NGPS is Ready.")
zmq_status_service.py
Add class-level signal: sequencer_ready_signal = pyqtSignal()
In _status_from_seq_seqstate(), add to state_map: "RUNNING", "STARTING", "STOPPING", "FAILED", "ABORTING". Emit sequencer_ready_signal on the READY transition (guard against re-emit: only fire when previous state was not already "idle")
Add a seq_broadcast branch in listen() and a _handle_seq_broadcast(data) handler that emits new_message_signal with the severity/message/source fields from the payload. This topic carries all narrative operator messages that previously came over UDP async
layout_service.py— status_map in create_system_status_group()Add color entries for: "running" (green), "starting" (blue), "stopping" (blue), "aborting" (orange), "failed" (red)