8000 Fix two importing issues by jenshannoschwalm · Pull Request #18134 · darktable-org/darktable · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix two importing issues #18134

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

Merged
merged 2 commits into from
Jan 3, 2025

Conversation

jenshannoschwalm
Copy link
Collaborator
@jenshannoschwalm jenshannoschwalm commented Jan 2, 2025

Only ask for original dng files via github

We ask for dng files with specific exif tags that are currently not supported in darktable.

As some software (like from adobe) possibly adds those tags we get warnings for such images, but

  • we simply don't care about those as we want data from camera vendor
  • that gives irrelevant feedback to user
  • it spoils the database with tags

So if we find some non-dng name in "Exif.Image.OriginalRawFileName" we ignore those images for backreporting.

Fixes #18109


Don't report G_IO_ERROR_CANCELLED in _import_add_file_callback()

As we currently do in _import_enum_callback() the G_IO_ERROR_CANCELLED condition is not a real error and should not be reported as such.

We ask for dng files with specific exif tags that are currently not supported in darktable.

As some software (like from adobe) possibly adds those tags we get warnings for such images, but
- we simply don't care about those as we want data from camera vendor
- that gives irrelevant feedback to user
- it spoils the database with tags

So if we find some non-dng name in "Exif.Image.OriginalRawFileName" we ignore those images for backreporting.
As we currently do in _import_enum_callback() the G_IO_ERROR_CANCELLED condition
is not a real error and should not be reported as such.
@jenshannoschwalm jenshannoschwalm added the bugfix pull request fixing a bug label Jan 2, 2025
@jenshannoschwalm jenshannoschwalm added this to the 5.0.1 milestone Jan 2, 2025
Copy link
Member
@TurboGit TurboGit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks!

@TurboGit TurboGit merged commit 302f357 into darktable-org:master Jan 3, 2025
6 checks passed
@TurboGit
Copy link
Member
TurboGit commented Jan 3, 2025

Merged in master and darktable-5.0.x branch.

@jenshannoschwalm jenshannoschwalm deleted the import_50_issues branch February 6, 2025 06:19
@fabiosirna fabiosirna mentioned this pull request Mar 31, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix pull request fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple "forward matrix in '...', please report via darktable github" Messages on Import
2 participants
0