Skip to content

Conversation

jantonguirao
Copy link
Contributor

Description

Enhanced documentation of CenterCropPad

Motivation and Context

It clarifies an ambiguous description on how the operator extracts the cropping window, and what padding value is used

Copy link

codecov bot commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.24%. Comparing base (1e3be88) to head (bd50f86).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6464   +/-   ##
=======================================
  Coverage   57.24%   57.24%           
=======================================
  Files         507      507           
  Lines       31438    31438           
  Branches     3540     3540           
=======================================
  Hits        17998    17998           
  Misses      12594    12594           
  Partials      846      846           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Joaquin Anton Guirao <janton@nvidia.com>
@xadupre xadupre added this pull request to the merge queue Oct 18, 2024
Merged via the queue into onnx:main with commit 0ffd1d5 Oct 18, 2024
34 checks passed
Copy link
Contributor

@fdwr fdwr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Thank you Joaquin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants