Skip to content
Snippets Groups Projects
Commit 261d904b authored by Olivier BICHLER's avatar Olivier BICHLER
Browse files

Clarified behavior of forwardDims()

parent c5f78302
No related branches found
No related tags found
No related merge requests found
......@@ -208,7 +208,12 @@ public:
/**
* @brief Compute dimensions of input/output Tensors for each Operator of the
* GraphView object's Nodes.
* GraphView object's Nodes, by calling Node::forwardDims().
* This function verifies the following conditions:
* - Every node will forwardDims() regardless of if dims were previously forwarded or not;
* - forwadDims() calls are made in node dependencies order, because if dims have changed
* at any point in the graph, it must de propagated correctly to all succeeding nodes;
* - It handles cyclic dependencies correctly (currently only induced by the Memorize_Op).
*/
bool forwardDims(const std::vector<std::vector<DimSize_t>>& dims = {}, bool allowDataDependency = false);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment