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:
Modified Graphview after Conv_1
tiling, before this MR:
Modified Graphview after Conv_1
tiling, with naming technique proposed in this MR:
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
Activity
added EnhancementStructure 🏗 label
assigned to @idealbuq
added 8 commits
-
b86bb756...68aaf9b7 - 6 commits from branch
eclipse/aidge:dev
- 628b9466 - Add explicit names setting to nodes created in the getConvHorizontalTiling() function.
- 6b6e4626 - Removed unnecessary comments in code.
-
b86bb756...68aaf9b7 - 6 commits from branch
enabled an automatic merge when all merge checks for 6b6e4626 pass
changed milestone to %aidge v0.6.0