Skip to content
Snippets Groups Projects

Add explicit names setting to nodes created in the HorizontalTiling recipe

Context

In the getConvHorizontalTiling() function used in the HorizontalTiling recipe, Concat and Slice nodes were not named.

It is right that a node is not meant to be defined by its name and it is thus not a mandatory attribute. However, having specified node names as shown to be useful for further graph manipulations, such as for memory mapping features.

Illustration:

Original Graphview before tiling:

Annotation 2025-03-11 111819.png

Modified Graphview after Conv_1 tiling, before this MR:

Annotation 2025-03-11 112027.png

Modified Graphview after Conv_1 tiling, with naming technique proposed in this MR:

Annotation 2025-03-11 153159.png

Modified files

  • aidge_core/src/recipes/HorizontalTiling.cpp, to explicit set names to Concat, Slice and Slice's Producers nodes.

Detailed major modifications

Added a generic name to Concat nodes:

 concat -> setName(concat->type());

Added a generic name to Slice nodes, while keeping track of the original (Conv) node they are linked to (i.e. node->name):

 slice -> setName(node->name() + '_' + slice->type() + '_' + std::to_string(i));

Minor changes

Added an underscore in the names of Slices' Producers nodes for readability:

auto startsNode = Producer(starts, slice->name() + "_" + sliceInputsNames[1]);

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading