Skip to content
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

0 phash for small resized images #4

Open
sirpy opened this issue Jul 26, 2017 · 2 comments
Open

0 phash for small resized images #4

sirpy opened this issue Jul 26, 2017 · 2 comments

Comments

@sirpy
Copy link

sirpy commented Jul 26, 2017

Thanks for the great package.
I've noticed that images that i resize to 200-350(width) get a resulting buffer filled with zeros. the original image or a resized image to width 500 get a normal phash buffer.
any ideas?
Thanks

@timhuff
Copy link
Owner

timhuff commented Jul 26, 2017

Hi sirpy. Thanks for the compliment. Unfortunately I haven't really looked at this code for about 3 years now and I'm swamped at work. I wish I could be of greater help but I won't have much free time for the next month or so. But if you want to create a pull request containing a simple failing test, that would enable me to at least give it a shot here in the next week or so. Is it any image in the range of 200-350 or is that just an estimate? If so, it might have to do with the number 255 - does it work for images of width 257? 253?

@sirpy
Copy link
Author

sirpy commented Aug 1, 2017

hey, yes i confirmed that its about the number 255.
from width 255 some images get 0 while for 256 they dont

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

No branches or pull requests

2 participants