-
Notifications
You must be signed in to change notification settings - Fork 2
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
ARG mux proposal #30
base: v1.0.x
Are you sure you want to change the base?
ARG mux proposal #30
Changes from all commits
48e2f7b
2ea056c
27407c3
9216f90
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,39 @@ | ||||||||||||||||||
{ | ||||||||||||||||||
"$schema": "http://json-schema.org/draft-04/schema#", | ||||||||||||||||||
"description": "Describes substream constraints", | ||||||||||||||||||
"title": "Constraint Set", | ||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||
"type": "object", | ||||||||||||||||||
"minProperties": 1, | ||||||||||||||||||
"properties": { | ||||||||||||||||||
"description": { | ||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Each constraint set uses URN attribute names, since the parameter constraints in each set are intended to be extensible, and the metadata attributes therefore also have URNs, like Here it probably does make sense to use "simple" attribute names, like |
||||||||||||||||||
"description": "Substream group description", | ||||||||||||||||||
"type": "string" | ||||||||||||||||||
}, | ||||||||||||||||||
"format": { | ||||||||||||||||||
"description": "Substream format", | ||||||||||||||||||
"type": "string", | ||||||||||||||||||
"anyOf": [ | ||||||||||||||||||
{ | ||||||||||||||||||
"enum": [ | ||||||||||||||||||
"urn:x-nmos:format:video", | ||||||||||||||||||
"urn:x-nmos:format:audio", | ||||||||||||||||||
"urn:x-nmos:format:data" | ||||||||||||||||||
] | ||||||||||||||||||
}, | ||||||||||||||||||
{ | ||||||||||||||||||
"pattern": "^urn:x-nmos:format:(?!mux$)" | ||||||||||||||||||
} | ||||||||||||||||||
] | ||||||||||||||||||
}, | ||||||||||||||||||
"count": { | ||||||||||||||||||
"$ref": "param_constraint_number.json" | ||||||||||||||||||
}, | ||||||||||||||||||
Comment on lines
+28
to
+30
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We've discussed that most/all mux Receivers are capable of discarding substreams they are not interested in. Therefore we started discussing whether a E.g. something like
Suggested change
|
||||||||||||||||||
"constraint_sets": { | ||||||||||||||||||
"description": "Substream constraint sets", | ||||||||||||||||||
"type": "array", | ||||||||||||||||||
"items": { | ||||||||||||||||||
"$ref": "constraint_set.json" | ||||||||||||||||||
} | ||||||||||||||||||
} | ||||||||||||||||||
} | ||||||||||||||||||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've discussed that in order to maintain BCP-004-01 v1.0 compatibility this cannot be
urn:x-nmos:cap:substreams
because of the pattern property below. But not clear whetherurn:x-nmos:substreams
name is precise enough.