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

Added assertion in Pop operator

parent 267609cb
No related tags found
2 merge requests!279v0.4.0,!252Partial fix for issue eclipse/aidge/aidge_core#167
Pipeline #59518 passed
...@@ -24,6 +24,7 @@ Aidge::Elts_t Aidge::Pop_ProdConso::getNbRequiredData(const Aidge::IOIndex_t inp ...@@ -24,6 +24,7 @@ Aidge::Elts_t Aidge::Pop_ProdConso::getNbRequiredData(const Aidge::IOIndex_t inp
assert(mOp.getRawInput(inputIdx) && "requires valid input"); assert(mOp.getRawInput(inputIdx) && "requires valid input");
const Pop_Op& op = dynamic_cast<const Pop_Op&>(mOp); const Pop_Op& op = dynamic_cast<const Pop_Op&>(mOp);
AIDGE_ASSERT(!op.getInput(inputIdx)->empty(), "Pop operator requires known, non-empty, input dims for scheduling. You might have an unresolved data dependency upstream in the computing graph.");
return Elts_t::DataElts(op.getInput(inputIdx)->size() return Elts_t::DataElts(op.getInput(inputIdx)->size()
/ op.getInput(inputIdx)->dims()[0]); / op.getInput(inputIdx)->dims()[0]);
} }
......
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