Skip to content
Snippets Groups Projects

Fix slice for out of bound start and end attributes

Merged Cyril Moineau requested to merge SliceFix into dev
1 unresolved thread
  • Fix #221 (closed)
  • Add getInputs and getOutputs method to OperatorTensor (very minor change I didn't wanted to create another MR just for it)

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
253 253 const DimIdx_t axis = this->axes()[i] >= 0 ?
254 254 static_cast<DimIdx_t>(this->axes()[i]) :
255 255 static_cast<DimIdx_t>(this->axes()[i] + static_cast<DimIdx_t>(getInput(0)->nbDims()));
256 const DimSize_t start = this->starts()[i] >= 0 ?
256 DimSize_t start = this->starts()[i] >= 0 ?
257 257 static_cast<DimSize_t>(this->starts()[i]) :
258 258 static_cast<DimSize_t>(this->starts()[i] + static_cast<DimSize_t>(getInput(0)->dims()[axis]));
259 const DimSize_t end = this->ends()[i] >= 0 ?
259 // Clamp start to the range [0, axis_dim]
260 start = std::max(static_cast<DimSize_t>(0), std::min(start, getInput(0)->dims()[axis]-1));
261
262 DimSize_t end = this->ends()[i] >= 0 ?
260 263 static_cast<DimSize_t>(this->ends()[i]) :
261 264 static_cast<DimSize_t>(this->ends()[i] + static_cast<DimSize_t>(getInput(0)->dims()[axis]));
265 // Clamp end to the range [0, axis_dim]
266 end = std::max(static_cast<DimSize_t>(0), std::min(end, getInput(0)->dims()[axis]-1));
  • Houssem ROUIS
  • Cyril Moineau mentioned in commit 3d9f7779

    mentioned in commit 3d9f7779

  • Cyril Moineau added 1 commit

    added 1 commit

    • 698594a6 - Add getInputs and getOuputs method for OperatorTensor.

    Compare with previous version

  • Houssem ROUIS
  • Cyril Moineau added 1 commit

    added 1 commit

    • 8c69c41b - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Houssem ROUIS approved this merge request

    approved this merge request

  • Cyril Moineau mentioned in commit 9dd41e85

    mentioned in commit 9dd41e85

  • Please register or sign in to reply
    Loading