Skip to content

Fix/rewrite data handling #35

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

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Conversation

haakonnessjoen
Copy link
Member

Should be much better and performant on handling a lot of incoming data, as well as handling ack/nak messages

@haakonnessjoen
Copy link
Member Author

@phillipivan could you please test this version against your Ultrix?

@phillipivan
Copy link
Collaborator

Sure, I will run it up this afternoon or tomorrow.

@phillipivan
Copy link
Collaborator

Are you happy for me to merge #34 in the meantime?

@haakonnessjoen
Copy link
Member Author

sure

@phillipivan
Copy link
Collaborator

@haakonnessjoen have tested against an ultrix and can confirm things are working as expected:

  • Pulling SRC & DST labels
  • Taking new routes
  • Tally of existing routes

Logs attached.

module-log-4_16_2025-11_31_59 AM.csv

haakonnessjoen and others added 9 commits April 16, 2025 09:08
* Reworked data processing
* Fixed issue with not getting labels if "Request supported commands" config option was disabled
* Fixed issue where labels were duplicated and in wrong order if router had labels on multiple levels
* Fixed initial label request (non-extended mode) to use the correct format according to specification
* Fixed issue where not all source labels were received, fixes issue #31
…ice supports tally dump and live tally updates. Added check for the correct matrix in crosspoint functions.
@haakonnessjoen
Copy link
Member Author

@phillipivan Does the Ultrix support tally dump? And does it send tally updates when routed elsewhere too? If so, could you test the new optional tally dump feature? (config option)

@phillipivan
Copy link
Collaborator

See from page 82:
https://documentation.rossvideo.com/files/Manuals/Routers/Ultrix/Ultrix%20User%20Guide%20(2101DR-004).pdf

It looks like thats a yes to (a) and to (b) I think so, but I can double check.

Happy to test the new build but will have limited time outside production tomorrow, so it may slip till after Easter if I run out of time.

I should have added to the prior testing notes, we never noticed any issues with the labels, or anything else previously, so from an operator perspective there was no change.

@phillipivan
Copy link
Collaborator

Confirming the tallies track when routes are taken from another control system (in this case Dashboard)

@phillipivan
Copy link
Collaborator

@haakonnessjoen testing will have to wait till next week sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants