-
Notifications
You must be signed in to change notification settings - Fork 463
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
TF change antialias to true #1348
Conversation
10 x abs both with antialiasing:
only pt with antialiasing:
|
observation over 30 images: straight pages: no visible changes @charlesmindee @odulcy-mindee wdyt ? failing test will be fixed by #1336 |
@felixdittrich92 I'm not sure to understand the benchmark. Why do you compare tf and torch with PIL and OpenCV ? Is there also antialiasing enabled by default in these frameworks when you downsize an image ? |
82fcbdb
to
d02c4fb
Compare
I compare the way how we load the image and the framwork which performs the resizing with and without antialising For more information: #402 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1348 +/- ##
=======================================
Coverage 95.76% 95.77%
=======================================
Files 155 155
Lines 6950 6951 +1
=======================================
+ Hits 6656 6657 +1
Misses 294 294
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Add the end what we want is to work on the same images in both frameworks in the whole pipe inference and training i have compared the "loading" libs and processing frameworks to see the difference :) The black images shows the smallest diff which are:
both with |
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.
Thanks !
This PR:
antialias
also for TF to TrueNOTE tests:
I don't think there's much we can do about that (the results are correct and close to PT see below)