-
Notifications
You must be signed in to change notification settings - Fork 24
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
Why do we need expand2sqaure in scorers? #44
Comments
In my survey, I found that no expandsqure can promote the capicity of whole model... |
hi, without expandsqure, do you directly resize to 224? I wonder whether directly resize would defect the original image. |
CLIPImageProcessor includes a CenterCrop and a resize, rather than directly resizing the image to a square shape. |
In my sight, I concern the padding part may infect the image quality in some way. |
Hello, thanks for your great works.
However, I find that the preprocess inside this looks weird.
The input image is first expanded to a square one, with extra mean-color margins by applying the expand2sqaure function.
I visualized a preprocessed image, which looks like this:
while the original image is:
My teammate also reports that larger aspect ratio results in lower aesthetic score. So we are wondering if this preprocess is correct.
The text was updated successfully, but these errors were encountered: