Skip to content

Conversation

@justinchuby
Copy link
Collaborator

@justinchuby justinchuby commented Sep 4, 2025

This is a follow up of #2540 to add a test described in #2539.

Fix #2539

@justinchuby justinchuby added the module: torchlib Related to the torch/aten function lib in development label Sep 4, 2025
@justinchuby justinchuby enabled auto-merge (squash) September 4, 2025 22:07
@codecov
Copy link

codecov bot commented Sep 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.99%. Comparing base (d98e3dd) to head (b54640b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2541   +/-   ##
=======================================
  Coverage   69.99%   69.99%           
=======================================
  Files         216      216           
  Lines       26074    26074           
  Branches     2618     2618           
=======================================
  Hits        18250    18250           
  Misses       6921     6921           
  Partials      903      903           

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

@justinchuby justinchuby disabled auto-merge September 4, 2025 22:16
@justinchuby justinchuby enabled auto-merge (squash) September 4, 2025 22:16
@justinchuby justinchuby added the merge at lgtm Reviewers can merge when they approve label Sep 4, 2025
@github-project-automation github-project-automation bot moved this from Todo to Done in ONNX Script Review Board Sep 4, 2025
@justinchuby justinchuby merged commit 1934901 into main Sep 4, 2025
32 checks passed
@justinchuby justinchuby deleted the justinchu/dynamic-padding-test branch September 4, 2025 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge at lgtm Reviewers can merge when they approve module: torchlib Related to the torch/aten function lib in development

Projects

Development

Successfully merging this pull request may close these issues.

Dynamic Padding bug, with a minimal fix!

3 participants