[fix-comments]: in/outFIFODepth renamed to in/outFIFODepths

src/finn/transformation/fpgadataflow/derive_characteristic.py

@@ -127,7 +127,7 @@ class DeriveCharacteristic(NodeLocalTransformation):
 class DeriveFIFOSizes(NodeLocalTransformation):
     """Prerequisite: DeriveCharacteristic already called on graph.
     For each node in the graph, use the accumulated I/O characteristic function
-    to perform FIFO sizing, setting the in/outFIFODepth attributes of HLSCustomOp
+    to perform FIFO sizing, setting the in/outFIFODepths attributes of HLSCustomOp
     nodes.
 
     * num_workers (int or None) number of parallel workers, see documentation in
@@ -178,7 +178,7 @@ class DeriveFIFOSizes(NodeLocalTransformation):
                     fifo_depth = int((prod_chrc_part - cons_chrc_part).max())
                     out_fifo_depths.append(fifo_depth)
                 # set output FIFO depth for this (producing) node
-                # InsertFIFO looks at the max of (outFIFODepth, inFIFODepth)
+                # InsertFIFO looks at the max of (outFIFODepths, inFIFODepths)
                 # for each tensor
                 prod.set_nodeattr("outFIFODepths", out_fifo_depths)
 

src/finn/transformation/fpgadataflow/insert_fifo.py

@@ -67,7 +67,7 @@ class InsertFIFO(Transformation):
     between fpgadataflow nodes.
 
     Takes the setting for the depth from the surrounding nodes by extracting
-    node attribute 'outFIFODepth' of the previous and node attribute 'inFIFODepth'
+    node attribute 'outFIFODepths' of the previous and node attribute 'inFIFODepths'
     of the subsequent node. max() of these two values sets the FIFO depth.
 
     Constructor arguments:
@@ -128,8 +128,8 @@ class InsertFIFO(Transformation):
                         folded output shape of the second node. A streaming fifo can't
                         be implemented in between these nodes."""
 
-                        # check if outFIFOdepth attribute of first node
-                        # and inFIFOdepth attribute of consumer node is equal
+                        # check if outFIFOdepths attribute of first node
+                        # and inFIFOdepths attribute of consumer node is equal
                         n0_depth = n0.get_nodeattr("outFIFODepths")[idx_out]
                         n1_depth = n1.get_nodeattr("inFIFODepths")[idx_inp]
 

src/finn/transformation/fpgadataflow/set_fifo_depths.py

@@ -201,7 +201,7 @@ class InsertAndSetFIFODepths(Transformation):
     Assumed input graph properties:
     - all nodes are fpgadataflow nodes
     - no FIFOs inserted,
-    - (inFIFODepth/outFIFODepth attrs will be ignored)
+    - (inFIFODepths/outFIFODepths attrs will be ignored)
 
     Output:
     - graph with appropriate-depth FIFOs inserted
@@ -216,7 +216,7 @@ class InsertAndSetFIFODepths(Transformation):
     - run through rtlsim with stream of multiple random input images (to fill pipeline)
     - keep track of observed maximum occupancy for each FIFO during rtlsim
     - when sim finished, update each FIFO depth to maximum observed occupancy
-      and set inFIFODepth/outFIFODepth attrs to 0 on relevant nodes
+      and set inFIFODepths/outFIFODepths attrs to 0 on relevant nodes
     """
 
     def __init__(
@@ -365,7 +365,7 @@ class InsertAndSetFIFODepths(Transformation):
             fifos[node.name] = sim[maxcount_name]
 
         # Apply depths back into the model;
-        # also set in/outFIFODepth to zero for non-FIFO
+        # also set in/outFIFODepths to zero for non-FIFO
         # nodes, preventing further FIFO insertion
         for node in model.graph.node:
             # set FIFO depth, reset FIFO implementation,