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.

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

Adding support for multicast stream data #788

Merged
merged 7 commits into from
Oct 12, 2023
Merged

Conversation

ryan-summers
Copy link
Member

@ryan-summers ryan-summers commented Oct 11, 2023

This PR fixes #786 by updating the python tools to direct Stabilizer to stream to multicast UDP addresses. Note that there is no firmware change required for this.

Local run of the streaming HITL:

$ python311 hitl/streaming.py --broker 10.35.20.1 dt/sinara/dual-iir/04-91-62-d2-a8-6f
INFO:gmqtt.mqtt.protocol:[CONNECTION MADE]
INFO:gmqtt.mqtt.package:[SEND SUB] 1 [b'dt/sinara/dual-iir/04-91-62-d2-a8-6f/response']
INFO:gmqtt.mqtt.package:[SEND SUB] 2 [b'dt/sinara/dual-iir/04-91-62-d2-a8-6f/settings/#']
INFO:gmqtt.mqtt.handler:[SUBACK] 1 (0,)
INFO:miniconf.miniconf:Handling subscription for 1
INFO:gmqtt.mqtt.handler:[SUBACK] 2 (0,)
INFO:miniconf.miniconf:Handling subscription for 2
INFO:__main__:Starting stream
INFO:miniconf.miniconf:Sending "{'ip': [239, 192, 1, 100], 'port': 9293}" to "dt/sinara/dual-iir/04-91-62-d2-a8-6f/settings/stream_target" with CD: b'9ea73129681e11ee979b60f262b6eb3e'
INFO:__main__:Testing stream reception
INFO:stabilizer.stream:Connection made (listening)
INFO:stabilizer.stream:Received 59.9231 MB, 5.99231 MB/s
INFO:stabilizer.stream:Loss: 38918/975216 batches (3.99071 %)
INFO:__main__:Stopping stream
INFO:miniconf.miniconf:Sending "{'ip': [0, 0, 0, 0], 'port': 0}" to "dt/sinara/dual-iir/04-91-62-d2-a8-6f/settings/stream_target" with CD: b'a49b9329681e11ee92d760f262b6eb3e'
INFO:__main__:Draining queue
INFO:__main__:Verifying no further frames are received
PASS

@ryan-summers ryan-summers requested a review from jordens October 11, 2023 10:11
@github-actions github-actions bot added the changes-hitl Changes HITL scripts, needs close review label Oct 11, 2023
Copy link
Member

@jordens jordens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good!

py/stabilizer/stream.py Outdated Show resolved Hide resolved
py/stabilizer/stream.py Outdated Show resolved Hide resolved
hitl/streaming.py Outdated Show resolved Hide resolved
py/stabilizer/stream.py Outdated Show resolved Hide resolved
hitl/streaming.py Outdated Show resolved Hide resolved
hitl/streaming.py Outdated Show resolved Hide resolved
@ryan-summers ryan-summers requested a review from jordens October 11, 2023 16:21
py/stabilizer/stream.py Outdated Show resolved Hide resolved
py/stabilizer/stream.py Show resolved Hide resolved
@ryan-summers ryan-summers requested a review from jordens October 12, 2023 10:21
hitl/streaming.py Show resolved Hide resolved
py/stabilizer/stream.py Outdated Show resolved Hide resolved
@jordens jordens added this pull request to the merge queue Oct 12, 2023
Merged via the queue into main with commit 43122c1 Oct 12, 2023
8 of 9 checks passed
@jordens jordens deleted the feature/multicast-streaming branch October 30, 2023 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes-hitl Changes HITL scripts, needs close review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stream to multicast
2 participants