Conversation
Collaborator
lo-simon
commented
Nov 14, 2025
lo-simon
commented
- Add IS-11 NMOS Stream Compatibility Management support.
- This is superseded IS-11 support #271 which was initially created by @N-Nagorny
…move Flow Compatibility API helper functions into details namespace
…raint Set is a subset of another one
…alue with debug message
…callback to parse "transport_file" and Stream Compatibility Management API callback to perform application-specific receiver validation with its transport file.
… be initialized after" warning
Comment on lines
+1061
to
+1063
| auto output = edid_support | ||
| ? nmos::experimental::make_streamcompatibility_output(output_id, device_id, true, boost::variant<utility::string_t, web::uri>(edid), receiver_ids, model.settings) | ||
| : nmos::experimental::make_streamcompatibility_output(output_id, device_id, true, receiver_ids, model.settings); |
Contributor
There was a problem hiding this comment.
Hmmm, what's the behaviour if EDID support is set to true in the config? Does this pass any of the EDID tests in the NMOS Testing tool?
| if (!nmos::caps::meta::enabled(constraint_set)) continue; | ||
| for (const auto& sender_caps_constraint_set : sender_capabilities) | ||
| { | ||
| const auto intersection = nmos::experimental::get_constraint_set_intersection(sender_caps_constraint_set, constraint_set); |
Contributor
There was a problem hiding this comment.
Should we worry about the Windows 2022 warning?
declaration of 'intersection' hides function parameter
Collaborator
Author
There was a problem hiding this comment.
Rename the variable intersection to constraint_set_intersection to prevent declaration of 'intersection' hides function parameter warning.
| } | ||
| return false; | ||
| } | ||
| return lhs < rhs; |
Contributor
There was a problem hiding this comment.
Are we guarding against a rational being compared with a scalar, or is that a legal comparison?
| } | ||
|
|
||
| // Constraint B is a subconstraint of Constraint A if: | ||
| // 1. Constraint B has enum keyword when Constraint A has it and enumerated values of Constraint B are a subset of enumerated values of Constraint A |
Contributor
There was a problem hiding this comment.
"Constraint B has enum keyword when Constraint also has it" perhaps?
Co-authored-by: jonathan-r-thorpe <64410119+jonathan-r-thorpe@users.noreply.github.com>
Co-authored-by: jonathan-r-thorpe <64410119+jonathan-r-thorpe@users.noreply.github.com>
…o prevent `declaration of 'intersection' hides function parameter` warning
# Conflicts: # Development/nmos-cpp-node/node_implementation.cpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.