From 5a924669b7d8946632f2f9ba1f2daa8c19054a74 Mon Sep 17 00:00:00 2001 From: Jiuqiang Tang Date: Mon, 20 Mar 2023 17:08:20 -0700 Subject: [PATCH] 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. PiperOrigin-RevId: 518117759 --- mediapipe/framework/tool/subgraph_expansion.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/mediapipe/framework/tool/subgraph_expansion.cc b/mediapipe/framework/tool/subgraph_expansion.cc index 9f81153f1..dcd055f59 100644 --- a/mediapipe/framework/tool/subgraph_expansion.cc +++ b/mediapipe/framework/tool/subgraph_expansion.cc @@ -183,12 +183,13 @@ absl::Status FindCorrespondingStreams( // name, calculator, input_stream, output_stream, input_side_packet, // output_side_packet, options. // 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( const CalculatorGraphConfig::Node& subgraph_node) { if (subgraph_node.source_layer() || subgraph_node.buffer_size_hint() || subgraph_node.has_output_stream_handler() || - subgraph_node.input_stream_info_size() != 0 || - !subgraph_node.executor().empty()) { + subgraph_node.input_stream_info_size() != 0) { return mediapipe::InvalidArgumentErrorBuilder(MEDIAPIPE_LOC) << "Subgraph \"" << subgraph_node.name() << "\" has a field that is only applicable to calculators.";