-
Notifications
You must be signed in to change notification settings - Fork 7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Negative padding for functional_pil #2381 #2744
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2744 +/- ##
==========================================
+ Coverage 73.05% 73.07% +0.01%
==========================================
Files 96 96
Lines 8298 8304 +6
Branches 1291 1292 +1
==========================================
+ Hits 6062 6068 +6
Misses 1838 1838
Partials 398 398
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jamt9000 thanks for the PR ! Looks good.
I left a comment to change the test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks @jamt9000
Along with pytorch#2744 this will make negative padding uniform between PIL and Tensor pytorch#2381
* Negative padding for functional_tensor symmetric Along with #2744 this will make negative padding uniform between PIL and Tensor #2381 * Enable tests for negative symmetric pad with tensor Co-authored-by: vfdev <[email protected]>
* Negative padding for functional_pil pytorch#2381 * Tests for PIL negative padding pytorch#2381 * Move PIL vs tensor test inside test_pad * Adapt test_pad from test_transforms_tensor.py Co-authored-by: vfdev <[email protected]>
* Negative padding for functional_tensor symmetric Along with pytorch#2744 this will make negative padding uniform between PIL and Tensor pytorch#2381 * Enable tests for negative symmetric pad with tensor Co-authored-by: vfdev <[email protected]>
* Negative padding for functional_pil pytorch#2381 * Tests for PIL negative padding pytorch#2381 * Move PIL vs tensor test inside test_pad * Adapt test_pad from test_transforms_tensor.py Co-authored-by: vfdev <[email protected]>
* Negative padding for functional_tensor symmetric Along with pytorch#2744 this will make negative padding uniform between PIL and Tensor pytorch#2381 * Enable tests for negative symmetric pad with tensor Co-authored-by: vfdev <[email protected]>
Fix as discussed in #2381 (tests to follow)