-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[image information]:More exif info fields #16461
Conversation
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.
Do not upgrade on my side:
1,6624 [init] can't rename images
1,6624 [init] error in view v_images: no such table: main.images
1,6629 [init] database `library.db' couldn't be upgraded from version 50 to 51. aborting
1,6635 ERROR : cannot open database
I think that as for version 41 -> 42 you need to remove v_images and create it back.
You may also want to squash some commits to have a clean history (last commit here could be merged into the database migration commit). |
- white balance - flash - exposure program - metering mode
- white balance - flash - exposure program - metering mode fix SQL error recreate view v_images
8a0718e
to
291065e
Compare
Fixed the v_images issue, squashed and rebased. |
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.
Works for me now. Thanks!
Need a release note entry.
For the release notes: Added a few more EXIF fields to the image information module: White balance, exposure program, flash and metering mode. |
Thanks! |
This PR adds a few more EXIF fields to the image info module:
This is filled automatically for newly imported images. For existing images, a refresh EXIF is required
closes #16418