fix optional inputs in node treated as input for the graph (Fix optional bias in Conv operator)

Context

Conv operators may have optional Bias. In this case, the missing inputs are treated as free available input of the GraphView.

Modified files

• modified: include/aidge/graph/Node.hpp
• modified: src/graph/Node.cpp

Detailed major modifications

• getNbFreeDataInputs : return only the number of available Data and DataOptional (not Params)
• getFirstFreeDataInput : return the first non assignated Data or DataOptional
• AddChild : Instead of getting the first node from the GraphView inputnode, it finds the first input node with data available (which should be unique)

include/aidge/graph/Node.hpp

@@ -217,10 +217,10 @@ public:
       if ((inputCategory(i) == InputCategory::Data || inputCategory(i) == InputCategory::OptionalData)
         && input(i).second == gk_IODefaultIndex)
       {
-        break;
+        return i;
       }
     }
-    return (i < nbInputs()) ? i : gk_IODefaultIndex;
+    return gk_IODefaultIndex;
   }
 
 
@@ -547,3 +547,14 @@ private:
 
 
 #endif /* AIDGE_CORE_GRAPH_NODE_H_ */
+/********************************************************************************
+ * Copyright (c) 2023 CEA-List
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License 2.0 which is available at
+ * http://www.eclipse.org/legal/epl-2.0.
+ *
+ * SPDX-License-Identifier: EPL-2.0
+ *
+ ********************************************************************************/
+