Skip to content

Make IS-08 Channel Mapping editable - #178

Merged
lo-simon merged 7 commits into
sony:masterfrom
garethsb:feature/channelmapping-edit
Aug 20, 2026
Merged

Make IS-08 Channel Mapping editable#178
lo-simon merged 7 commits into
sony:masterfrom
garethsb:feature/channelmapping-edit

Conversation

@garethsb

@garethsb garethsb commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add Device Edit for Channel Mapping: local draft on Active Map, Activate posts only the changed channels through dataProvider UPDATE of devices.
  • Soft-validate routable_inputs, block_size and reordering on the matrix without blocking Activate, so a Node can still be tested with a predicted-invalid POST.
  • Add scheduled activation modes (requested_time) matching Connection Edit, a pending Activations show tab, and Cancel via DELETE …/map/activations/{id}.

Test plan

  • Immediate Activate against nmos-cpp-node updates Active Map and a later Edit seeds from that map.
  • Mapping cells that violate routable_inputs / incomplete blocks / non-fixed offset show the highest-priority warning; Activate still posts.
  • Relative schedule with a long requested_time (e.g. 300:0) appears on Activations and can be cancelled.
  • No Bridge / Auto Bridge / Forced Bridge still use the resolved $channelmappingAPI.
  • yarn test in Development/ (Jest now fails if no tests are found).

Document Show/Edit (not API staged), diff-only activations with scheduled
modes, soft cap warnings, and a separate Activations tab.
Open Edit from the Active Map tab, keep a local draft, and POST only the
changed channels through dataProvider UPDATE of devices. Require Jest to
find tests now that the first suite exists.

Signed-off-by: Gareth Sylvester-Bradley <garethsb@nvidia.com>
Keep every mapping control clickable. Unselected cells that the Output
does not list use a faded warning colour; selected ones keep the check
in the same warning colour so a Node can still be tested with a POST.

Signed-off-by: Gareth Sylvester-Bradley <garethsb@nvidia.com>
Predict incomplete blocks and non-fixed offsets from the draft map, then
show the highest-priority warning on selected cells. Leave Activate enabled.

Signed-off-by: Gareth Sylvester-Bradley <garethsb@nvidia.com>
POST relative and absolute modes with requested_time, list pending
activations from $activations, and cancel them with DELETE.

Signed-off-by: Gareth Sylvester-Bradley <garethsb@nvidia.com>
Signed-off-by: Gareth Sylvester-Bradley <garethsb@nvidia.com>
Use a non-interactive MappingCellTooltip on matrix cells so the popover
does not block the cell below or stay open after the pointer leaves.
Keep MappingHeadTooltip interactive for header names and chip links.

Signed-off-by: Gareth Sylvester-Bradley <garethsb@nvidia.com>

@lo-simon lo-simon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@lo-simon
lo-simon merged commit 182b372 into sony:master Aug 20, 2026
3 checks passed
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