Minor changes

include/aidge/graph/GraphView.hpp

@@ -98,6 +98,11 @@ public:
      */
     void save(std::string path, bool verbose = false, bool showProducers = true) const;
 
+    /**
+     * Check that a node is in the current GraphView.
+     * @param nodePtr Node to check
+     * @return bool True is nodePtr belongs to the GraphView.
+    */
     inline bool inView(NodePtr nodePtr) const {
         return mNodes.find(nodePtr) != mNodes.end();
     }

src/graph/GraphView.cpp

@@ -401,6 +401,8 @@ void Aidge::GraphView::setInputId(Aidge::IOIndex_t /*inID*/,
 }
 
 void Aidge::GraphView::add(std::shared_ptr<Node> node, bool includeLearnableParam) {
+  AIDGE_ASSERT(node != nullptr, "Trying to add non-existant node!");
+
   // first node to be added to the graph is the root node by default
   if (mRootNode == nullptr) {
     mRootNode = node;
@@ -442,7 +444,7 @@ std::pair<std::vector<Aidge::NodePtr>, size_t> Aidge::GraphView::getRankedNodes(
 
     for (auto childs : curNode->getOrderedChildren()) {
       for (auto child : childs) {
-        if (nodesToRank.find(child) != nodesToRank.end()) {
+        if (child != nullptr && nodesToRank.find(child) != nodesToRank.end()) {
           rankedNodes.push_back(child);
           nodesToRank.erase(child);
         }
@@ -450,7 +452,7 @@ std::pair<std::vector<Aidge::NodePtr>, size_t> Aidge::GraphView::getRankedNodes(
     }
 
     for (auto parent : curNode->getParents()) {
-      if (nodesToRank.find(parent) != nodesToRank.end()) {
+      if (parent != nullptr && nodesToRank.find(parent) != nodesToRank.end()) {
         rankedNodes.push_back(parent);
         nodesToRank.erase(parent);
       }
@@ -538,7 +540,7 @@ bool Aidge::GraphView::add(std::set<std::shared_ptr<Node>> otherNodes, bool incl
 
     for (auto childs : curNode->getOrderedChildren()) {
       for (auto child : childs) {
-        if (nodesToRank.find(child) != nodesToRank.end()) {
+        if (child != nullptr && nodesToRank.find(child) != nodesToRank.end()) {
           rankedNodes.push_back(child);
           nodesToRank.erase(child);
 
@@ -551,7 +553,7 @@ bool Aidge::GraphView::add(std::set<std::shared_ptr<Node>> otherNodes, bool incl
     }
 
     for (auto parent : curNode->getParents()) {
-      if (nodesToRank.find(parent) != nodesToRank.end()) {
+      if (parent != nullptr && nodesToRank.find(parent) != nodesToRank.end()) {
         rankedNodes.push_back(parent);
         nodesToRank.erase(parent);
 

src/operator/OperatorTensor.cpp

@@ -152,6 +152,7 @@ void Aidge::OperatorTensor::setDataType(const DataType& dataType) const {
     }
 
     for (IOIndex_t i = nbData(); i < nbInputs(); ++i) {
+        AIDGE_ASSERT(getInput(i) != nullptr, "Missing input#{} for operator {}", i, type());
         getInput(i)->setDataType(dataType);
     }
 }
\ No newline at end of file