moar

include/aidge/graph/GraphView.hpp

@@ -394,7 +394,7 @@ public:
     std::pair<std::vector<NodePtr>, size_t> getRankedNodes() const;
 
     /**
-    // todo see comment in code
+    // todo comment is in code
      * Get the nodes name according to the GraphView nodes ranking.
      * @param format The formatting string to be used with fmt::format().
      * @details The usable positional arguments are the following:

src/graph/GraphView.cpp

@@ -762,7 +762,7 @@ std::map<Aidge::NodePtr, std::string> Aidge::GraphView::getRankedNodesName(const
     std::map<std::string, size_t>::iterator it;
     std::tie(it, std::ignore) = typeRank.insert(std::make_pair(rankedNode->type(), 0));
 
-    const auto name = (markNonUnicity && rank < rankedNodes.second)  //if todo logic doesn't match description, to be checked
+    const auto name = (markNonUnicity && rank < rankedNodes.second)  //if todo logic doesn't match description, to be checked, i'd say > and swap ? and :
       ? fmt::format(format, rankedNode->name(), rankedNode->type(), rank, it->second)  //then
       : fmt::format(format, rankedNode->name(), rankedNode->type(), fmt::format("?{}", rank), fmt::format("?{}", it->second));  //else
     rankedNodesName.insert(std::make_pair(rankedNode, name));