cassdlcm/mediapipe
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Temporarily disabling checking whether the executor is not set in a subgraph node. This is a workaround to allow the MediaPipe InferenceCalculator to have its own executor.

Browse Source 
PiperOrigin-RevId: 518117759
This commit is contained in:
Jiuqiang Tang 2023-03-20 17:08:20 -07:00 committed by Copybara-Service
parent 2651d30ebf
commit 5a924669b7
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
mediapipe/framework/tool/subgraph_expansion.cc
View File

@ -183,12 +183,13 @@ absl::Status FindCorrespondingStreams(
// name, calculator, input_stream, output_stream, input_side_packet, // name, calculator, input_stream, output_stream, input_side_packet,
// output_side_packet, options. // output_side_packet, options.
// All other fields are only applicable to calculators. // All other fields are only applicable to calculators.
// TODO: Check whether executor is not set in the subgraph node
// after this issues is properly solved.
absl::Status ValidateSubgraphFields( absl::Status ValidateSubgraphFields(
const CalculatorGraphConfig::Node& subgraph_node) { const CalculatorGraphConfig::Node& subgraph_node) {
if (subgraph_node.source_layer() || subgraph_node.buffer_size_hint() || if (subgraph_node.source_layer() || subgraph_node.buffer_size_hint() ||
subgraph_node.has_output_stream_handler() || subgraph_node.has_output_stream_handler() ||
subgraph_node.input_stream_info_size() != 0 || subgraph_node.input_stream_info_size() != 0) {
!subgraph_node.executor().empty()) {
return mediapipe::InvalidArgumentErrorBuilder(MEDIAPIPE_LOC) return mediapipe::InvalidArgumentErrorBuilder(MEDIAPIPE_LOC)
<< "Subgraph \"" << subgraph_node.name() << "Subgraph \"" << subgraph_node.name()
<< "\" has a field that is only applicable to calculators."; << "\" has a field that is only applicable to calculators.";