AMWA NMOS IS-05 1.2-dev / MXL Support by jonathan-r-thorpe · Pull Request #483 · sony/nmos-cpp

jonathan-r-thorpe · 2026-03-05T09:58:55Z

Prototype used to demonstrate MXL integration with NMOS

…e and use "auto" in /staged)

…id-auto Add support for "auto" to MXL sender flow_id transport parameter

…nnections Renamed flow_id to mxl_flow_id in MXL sender and receiver connections

… of BCP-007-03

garethsb

Forgot to submit this last week, sorry...

garethsb · 2026-05-15T12:36:14Z

-    //"senders": ["v", "a"],
-    //"receivers": [],
+    //"senders": ["v", "a", "d"],
+    //"receivers": ["v", "a", "d"],


Some of the comment examples in this file show what the equivalent of the default would be, some show interesting non-default examples. This was the latter (only create senders, don't create any receivers), now it's the former. Change made intentionally?

garethsb · 2026-05-15T12:37:30Z


-    // video_type: media type of video flows, e.g. "video/raw" or "video/jxsv", see nmos::media_types
-    //"video_type": "video/jxsv",
+    // video_type: media type of video flows, e.g. "video/raw", "video/jxsv", or "video/v210" for MXL Senders and Receivers, see nmos::media_types


Suggested change

// video_type: media type of video flows, e.g. "video/raw", "video/jxsv", or "video/v210" for MXL Senders and Receivers, see nmos::media_types

// video_type: media type of video flows, e.g. "video/raw", "video/jxsv" for ST 2110 senders and receivers

// or "video/v210" for MXL senders and receivers, see nmos::media_types

garethsb · 2026-05-15T12:38:54Z


    // senders, receivers: controls which kinds of sender and receiver are instantiated by the example node
    // the values must be an array of unique strings identifying the kinds of 'port', like ["v", "a", "d"], see impl::ports
+    // for MXL Senders and Receivers, the values must be ["xv", "xa", "xd"]


Although in docs we normally Title Case the NMOS keywords, that seems not to be the case in this file. Let's not end up with a mix anyway?

Suggested change

// for MXL Senders and Receivers, the values must be ["xv", "xa", "xd"]

// for MXL senders and receivers, the values must be like ["xv", "xa", "xd"]

garethsb · 2026-05-15T12:44:41Z

+                // BCP-007-03: PATCH /staged for MXL receivers must not contain transport_file.
+                // See https://specs.amwa.tv/bcp-007-03/branches/publish-auto-null/docs/NMOS-With-MXL.html


@jonathan-r-thorpe I think this is a mistake in the spec? Given that the response schema requires that transport_file is present, I think it should be valid to stage:

"transport_file": { "type": null, "data": null }

The BCP-007-03 spec is ambiguous on this so I'll raise it as an issue on the spec.
BCP-007-03 says PATCH /staged for MXL receivers is “not expected to contain” transport_file, and that controllers “MUST NOT provide” the attribute — which I read as prohibiting the key entirely. That’s stricter than the IS-05 stage schema, as you point out. I think this might become moot if the position on transport_file changes in the spec...

garethsb · 2026-05-18T09:11:28Z

 {
    namespace is05_schemas
    {
+        namespace v1_2_x


Suggested change

namespace v1_2_x

namespace v1_2_dev

garethsb · 2026-05-18T09:16:56Z

+    const nmos::id mxl_domain_id = model.settings.has_field(impl::fields::mxl_domain_id)
+        ? impl::fields::mxl_domain_id(model.settings)
+        : nmos::make_repeatable_id(seed_id, U("/x-nmos/mxl/domain"));


Does it make sense to provide a default for this? Doing so seems to imply it would be reasonable for a media function to create its own domain?

garethsb · 2026-05-18T09:19:15Z

+                // add optional grain_rate
+                flow.data[nmos::fields::grain_rate] = nmos::make_rational(frame_rate);


Given MXL moves us even further from SDI than ST 2022-6 and ST 2110, maybe we drop the (video) grain rate on audio?

garethsb · 2026-05-18T09:23:21Z

+                flow = nmos::make_raw_audio_flow(flow_id, source_id, device_id, 48000, 32, model.settings);
+                flow.data[nmos::fields::media_type] = value::string(U("audio/float32"));


⬇️ Maybe we should use make_audio_flow or have another overload of make_raw_audio_flow, so we don't have to rewrite the media_type?

garethsb · 2026-05-18T09:24:44Z

+                if (nmos::media_types::video_raw == video_type)
+                {
+                    const auto interlace_modes = nmos::interlace_modes::progressive != interlace_mode
+                        ? std::vector<utility::string_t>{ nmos::interlace_modes::interlaced_bff.name, nmos::interlace_modes::interlaced_tff.name, nmos::interlace_modes::interlaced_psf.name }
+                        : std::vector<utility::string_t>{ nmos::interlace_modes::progressive.name };
+                    receiver.data[nmos::fields::caps][nmos::fields::constraint_sets] = value_of({
+                        value_of({
+                            { nmos::caps::format::grain_rate, nmos::make_caps_rational_constraint({ frame_rate }) },
+                            { nmos::caps::format::frame_width, nmos::make_caps_integer_constraint({ frame_width }) },
+                            { nmos::caps::format::frame_height, nmos::make_caps_integer_constraint({ frame_height }) },
+                            { nmos::caps::format::interlace_mode, nmos::make_caps_string_constraint(interlace_modes) },
+                            { nmos::caps::format::color_sampling, nmos::make_caps_string_constraint({ sampling.name }) }
+                        })
+                    });
+                }
+                else if (nmos::media_types::video_jxsv == video_type)
+                {
+                    const auto max_format_bit_rate = nmos::get_video_jxsv_bit_rate(frame_rate, frame_width, frame_height, max_bits_per_pixel);
+                    const auto max_transport_bit_rate = uint64_t(transport_bit_rate_factor * max_format_bit_rate / 1e3 + 0.5) * 1000;
+
+                    receiver.data[nmos::fields::caps][nmos::fields::constraint_sets] = value_of({
+                        value_of({
+                            { nmos::caps::format::profile, nmos::make_caps_string_constraint({ profile.name }) },
+                            { nmos::caps::format::level, nmos::make_caps_string_constraint({ level.name }) },
+                            { nmos::caps::format::sublevel, nmos::make_caps_string_constraint({ nmos::sublevels::Sublev3bpp.name, nmos::sublevels::Sublev4bpp.name }) },
+                            { nmos::caps::format::bit_rate, nmos::make_caps_integer_constraint({}, nmos::no_minimum<int64_t>(), (int64_t)max_format_bit_rate) },
+                            { nmos::caps::transport::bit_rate, nmos::make_caps_integer_constraint({}, nmos::no_minimum<int64_t>(), (int64_t)max_transport_bit_rate) },
+                            { nmos::caps::transport::packet_transmission_mode, nmos::make_caps_string_constraint({ nmos::packet_transmission_modes::codestream.name }) }
+                        })
+                    });
+                }


Needs fixing to work with video/v210 and maybe video/v210a.

garethsb · 2026-05-18T09:27:22Z

+            }
+            else if (impl::ports::mxl_audio == port)
+            {
+                receiver = nmos::make_receiver(receiver_id, device_id, nmos::transports::mxl, {}, nmos::formats::audio, { nmos::media_type{ U("audio/float32") } }, model.settings);


We should add an nmos::media_types::audio_float32 constant? (And the MXL video ones too.)

garethsb and others added 18 commits February 25, 2026 10:08

Start with v1.1.x schemas

33c9902

Add schema changes for MXL as a built-in transport

7f4d4a6

Add IS-05 schema cpp generation

f28ca6a

Add IS-05 v1.2 as copy of IS-05 v1.1

d0c600e

Add MXL senders and receivers

19db280

Fix no_auto_constraints

de4b223

nmos-cpp-node support for MXL senders and receivers

5f8b468

Add flow_id as initial value (next, try making it a /constraints valu…

9fa1650

…e and use "auto" in /staged)

Add support for "auto" to MXL sender flow_id transport parameter

18a9097

Merge pull request #1 from jonathan-r-thorpe/feature/mxl-sender-flow_…

e175986

…id-auto Add support for "auto" to MXL sender flow_id transport parameter

Fix audio sender

e5c0105

update config.json to include mxl ports

a8b7535

Added mxl_domain_id transport parameter to MXL sender and receiver co…

cfb3726

…nnections Renamed flow_id to mxl_flow_id in MXL sender and receiver connections

Add missing JSON fields for MXL sender and receiver connections

4fb707f

Make mxl domain id configurable

d36cf53

Added mxl_domain_id to config.json

b4389a7

Ensure consistent mxl_domain_id and mxl_flow_id ordering

d2f8dbf

Update MXL configuration and implementation according to latest draft…

6a31cab

… of BCP-007-03

garethsb suggested changes May 18, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

AMWA NMOS IS-05 1.2-dev / MXL Support#483

AMWA NMOS IS-05 1.2-dev / MXL Support#483
jonathan-r-thorpe wants to merge 18 commits into
sony:masterfrom
jonathan-r-thorpe:nmos-mxl-test

jonathan-r-thorpe commented Mar 5, 2026

Uh oh!

garethsb left a comment

Uh oh!

garethsb May 15, 2026

Uh oh!

garethsb May 15, 2026

Uh oh!

garethsb May 15, 2026

Uh oh!

garethsb May 15, 2026

Uh oh!

jonathan-r-thorpe May 18, 2026

Uh oh!

garethsb May 18, 2026

Uh oh!

garethsb May 18, 2026

Uh oh!

garethsb May 18, 2026

Uh oh!

garethsb May 18, 2026

Uh oh!

garethsb May 18, 2026

Uh oh!

garethsb May 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	// video_type: media type of video flows, e.g. "video/raw", "video/jxsv", or "video/v210" for MXL Senders and Receivers, see nmos::media_types
	// video_type: media type of video flows, e.g. "video/raw", "video/jxsv" for ST 2110 senders and receivers
	// or "video/v210" for MXL senders and receivers, see nmos::media_types

	// for MXL Senders and Receivers, the values must be ["xv", "xa", "xd"]
	// for MXL senders and receivers, the values must be like ["xv", "xa", "xd"]

		// BCP-007-03: PATCH /staged for MXL receivers must not contain transport_file.
		// See https://specs.amwa.tv/bcp-007-03/branches/publish-auto-null/docs/NMOS-With-MXL.html

		// add optional grain_rate
		flow.data[nmos::fields::grain_rate] = nmos::make_rational(frame_rate);

		flow = nmos::make_raw_audio_flow(flow_id, source_id, device_id, 48000, 32, model.settings);
		flow.data[nmos::fields::media_type] = value::string(U("audio/float32"));

Conversation

jonathan-r-thorpe commented Mar 5, 2026

Uh oh!

garethsb left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants