-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add support for .heic images #348
Comments
I also tried settings this in my nextcloud config but when I then run the background job the heic images are not processed.
|
Hi @ntimo |
Hey there, is this still something that could be added? Most recent phones nowadays have HEIC supported and like mine have it on by default, I actually spent some time trying to debug this 'issue' but I believe it is simply because it is still not supported |
Wow, it took me two years, but I finally got it done. With this, soon a version 1.0. 😬 |
I wouls like to personally say THANK YOU! |
Expected behaviour
.heic images should be scanned and analysed for faces.
Actual behaviour
.heic images are not analysed
Steps to reproduce
Additional Info
It seems to be related to this line there
facerecognition/lib/Service/SettingsService.php
Line 118 in b26f39a
Heic mime type
image/heic
iOS devices capture photos in the heic format, so having support for it would be huge. (https://support.apple.com/en-us/HT207022)
By the way, the app is amazing thanks for creating it.
The text was updated successfully, but these errors were encountered: