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

Mixed 2022-6/2110 inputs are not supported #211

Open
kierank opened this issue Dec 15, 2023 · 6 comments
Open

Mixed 2022-6/2110 inputs are not supported #211

kierank opened this issue Dec 15, 2023 · 6 comments

Comments

@kierank
Copy link

kierank commented Dec 15, 2023

2022-6 is type "mux" and 2110 is "video". This means than an input can't be selectable as either 2022-6 or 2110 in the real world.

Duplicating our UUIDs and making them codependent to solve this problem is unworkable (e.g what to do if user sets master_enable on both)

Allow 2022-6 to be of type video as by definition it will contain video.

@garethsb
Copy link
Contributor

e.g what to do if user sets master_enable on both

Reject the PATCH that sets a second master_enable --> true?

@kierank
Copy link
Author

kierank commented Dec 15, 2023

e.g what to do if user sets master_enable on both

Reject the PATCH that sets a second master_enable --> true?

So then it requires parking (master_enable=false) boths inputs to be able to switch over?

What vendors (e.g GV) have done in reality is make 2022-6 a video and it's by far a better solution. Forcing 2022-6 mux is doing so purely in the name of intellectual purity.

@garethsb
Copy link
Contributor

So then it requires parking (master_enable=false) boths inputs to be able to switch over?

Or go the other way and accept the PATCH and at activation, set master_enable to false automatically on the other one...

But...

What vendors (e.g GV) have done in reality is make 2022-6 a video and it's by far a better solution. Forcing 2022-6 mux is doing so purely in the name of intellectual purity.

Agreed... it's worth the experiment whether the current spec would allow, and existing Controllers would handle, connecting:

  • a Sender associated with a Flow compliant with video_coded.json schema, with format of video and media_type of video/SMPTE2022-6
  • a Receiver with format of video and media_types including video/SMPTE2022-6

I think they might.

@kierank
Copy link
Author

kierank commented Dec 15, 2023

So then it requires parking (master_enable=false) boths inputs to be able to switch over?

Or go the other way and accept the PATCH and at activation, set master_enable to false automatically on the other one...

But...

What vendors (e.g GV) have done in reality is make 2022-6 a video and it's by far a better solution. Forcing 2022-6 mux is doing so purely in the name of intellectual purity.

Agreed... it's worth the experiment whether the current spec would allow, and existing Controllers would handle, connecting:

* a Sender associated with a Flow compliant with _video_coded.json_ schema, with format of video and media_type of video/SMPTE2022-6

* a Receiver with format of video and media_types including video/SMPTE2022-6

I think they might.

Either the logic is in the controller or in the node but this codependency needs to be done somewhyere.

@garethsb
Copy link
Contributor

One question: would your Receiver, when in 'video/SMPTE2022-6' mode, be doing something with the audio and ancillary data It received?

@kierank
Copy link
Author

kierank commented Dec 19, 2023 via email

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

No branches or pull requests

2 participants