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

IS-05 Requirements for NMOS Controllers #148

Conversation

jonathan-r-thorpe
Copy link
Contributor

Addition of section for requirements specific to NMOS Controllers, based on the work done in the AWMA NMOS Controller Specification meetings.

@jonathan-r-thorpe jonathan-r-thorpe changed the title IS-05 Requirements for NMOS Contollers IS-05 Requirements for NMOS Controllers Mar 2, 2022
@jonathan-r-thorpe jonathan-r-thorpe marked this pull request as ready for review March 16, 2022 19:19
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
garethsb and others added 4 commits April 12, 2022 14:04
Co-authored-by: Peter Brightwell <peter.brightwell@bbc.co.uk>
Co-authored-by: cristian-recoseanu <cristian.recoseanu@hotmail.com>
docs/Controllers.md Outdated Show resolved Hide resolved
garethsb and others added 2 commits April 19, 2022 12:27
Co-authored-by: cristian-recoseanu <cristian.recoseanu@hotmail.com>
docs/Controllers.md Show resolved Hide resolved
docs/Controllers.md Show resolved Hide resolved
docs/Controllers.md Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Show resolved Hide resolved
docs/Controllers.md Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
docs/Controllers.md Outdated Show resolved Hide resolved
@garethsb garethsb force-pushed the jonathan-r-thorpe-nmos-controller branch from d764672 to f015123 Compare May 11, 2022 10:05
Copy link
Contributor

@garethsb garethsb left a comment

Choose a reason for hiding this comment

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

LGTM 🤞

garethsb added a commit to garethsb/nmos-device-connection-management that referenced this pull request May 12, 2022
Based on AMWA-TV#148

Co-authored-by: Jonathan Thorpe (Sony) <Jonathan.Thorpe@sony.com>
Co-authored-by: Peter Brightwell <peter.brightwell@bbc.co.uk>
Co-authored-by: cristian-recoseanu <cristian.recoseanu@hotmail.com>
garethsb added a commit to garethsb/nmos-device-connection-management that referenced this pull request May 12, 2022
Based on AMWA-TV#148

Co-authored-by: Jonathan Thorpe (Sony) <Jonathan.Thorpe@sony.com>
Co-authored-by: Peter Brightwell <peter.brightwell@bbc.co.uk>
Co-authored-by: cristian-recoseanu <cristian.recoseanu@hotmail.com>
@garethsb
Copy link
Contributor

Approved and squashed for final review as #154. Congratulations and thank you, @jonathan-r-thorpe and the Controller Specification activity group.

@garethsb garethsb closed this May 12, 2022
peterbrightwell added a commit that referenced this pull request Jun 23, 2022
* Publish NMOS Controller

Based on #148

Co-authored-by: Jonathan Thorpe (Sony) <Jonathan.Thorpe@sony.com>
Co-authored-by: Peter Brightwell <peter.brightwell@bbc.co.uk>
Co-authored-by: cristian-recoseanu <cristian.recoseanu@hotmail.com>

* Clarify required Controller behaviour w.r.t. Receiver sender_id and Sender receiver_id
Change "SHOULD" to "can" in related Interop IS-04 paragraph, since a Node has no way to verify and enforce this

* Clarify sentence

* Apply suggestions from review

* Remove sentence whose meaning is now captured more precisely in the previous section

* Add missing statement about disconnect using IS-04 /target

* Remove redundant sentences.

* More minimal

* Consistent language

* Paragraph break

* tweak language

* Fix link

Co-authored-by: Peter Brightwell <peter.brightwell@bbc.co.uk>

Co-authored-by: Jonathan Thorpe (Sony) <Jonathan.Thorpe@sony.com>
Co-authored-by: Peter Brightwell <peter.brightwell@bbc.co.uk>
Co-authored-by: cristian-recoseanu <cristian.recoseanu@hotmail.com>
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.

4 participants