Skip to content
Snippets Groups Projects

[Upd] Shape so that 'forwardDims()' does not require a backend anymore

Merged Cyril Moineau requested to merge shapeForwardDims into dev
2 unresolved threads

Update shape so that forwardDims does not require a backend. Update test to remove unecessary Node.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
69 73 AIDGE_ASSERT(roi> 1, "Invalid ROI for Shape");
70 74
71 75 mOutputs[0]->resize({roi});
76 if (!mOutputs[0]->getImpl()){
77 Log::debug("Shape::forwardDims, no implementation set for output, defaulting to CPU.");
78 mOutputs[0]->setBackend("cpu");
  • 42 42 {1, 2, 3, 5}
    43 43 });
    44 44
    45 std::shared_ptr<Node> myShape = Shape();
    46 auto op = std::static_pointer_cast<OperatorTensor>(myShape -> getOperator());
    47 op->associateInput(0,input);
    45 std::shared_ptr<Shape_Op> op = std::make_shared<Shape_Op>();
    46 op->associateInput(0, input);
    48 47 op->setDataType(DataType::Int32);
    49 48 op->setBackend("cpu");
    50 myShape->forward();
    49 op->forward();
  • Maxence Naud approved this merge request

    approved this merge request

  • Maxence Naud changed title from Update Shape so that forwardDims does not require a backend. to [Upd] Shape so that 'forwardDims()' does not require a backend anymore

    changed title from Update Shape so that forwardDims does not require a backend. to [Upd] Shape so that 'forwardDims()' does not require a backend anymore

  • Olivier BICHLER approved this merge request

    approved this merge request

  • Olivier BICHLER added 12 commits

    added 12 commits

    • 8e07a49f...30d5f6e2 - 11 commits from branch dev
    • 3672c60e - Update shape so that forwardDims does not require a backend. Update test to remove unecessary Node.

    Compare with previous version

  • Olivier BICHLER enabled an automatic merge when all merge checks for 3672c60e pass

    enabled an automatic merge when all merge checks for 3672c60e pass

  • mentioned in commit 25b1ae50

  • Please register or sign in to reply
    Loading