From c59c6c8902af4abadeb2a7b5009c5a79cd61b739 Mon Sep 17 00:00:00 2001 From: markknoffler Date: Sat, 18 Apr 2026 16:45:00 +0530 Subject: [PATCH] [tmva][sofie] Fix ONNX fusion child indexing nodesChildren is keyed by original graph node indices, while the parse loop iterates over nodesOrder positions. Use nodesOrder[i] to retrieve the correct child list for the currently parsed node and avoid incorrect fusion pairing on reordered graphs. --- tmva/sofie_parsers/src/RModelParser_ONNX.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tmva/sofie_parsers/src/RModelParser_ONNX.cxx b/tmva/sofie_parsers/src/RModelParser_ONNX.cxx index aa444c05d5f76..aa196c510ad60 100644 --- a/tmva/sofie_parsers/src/RModelParser_ONNX.cxx +++ b/tmva/sofie_parsers/src/RModelParser_ONNX.cxx @@ -873,7 +873,7 @@ void RModelParser_ONNX::ParseONNXGraph(RModel & rmodel, const onnx::GraphProto & std::cout << "\t" << i << " " << nodesOrder[i] << " parsing operator " << op_type << std::endl; } - std::unique_ptr op = ParseOperator(i, graph, nodesOrder, nodesChildren[i]); + std::unique_ptr op = ParseOperator(i, graph, nodesOrder, nodesChildren[nodesOrder[i]]); if (!op) { if (verbose) { std::cout << "\t\tskipping operator since it is fused with previous one" << std::endl;