-
Notifications
You must be signed in to change notification settings - Fork 81
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
Date/Time Metadata incorrrect #507
Comments
Can you please try the dev tag of the docker image (webreaper/damselfly:dev) and refresh the metadata for the image, and let me know if that fixes it. If not, please can you attach a sample image so I can debug? |
Sure, I'll work on it this evening
…On Sun, Jan 7, 2024, 3:48 PM Mark Otway ***@***.***> wrote:
Can you please try the dev tag of the docker image
(webreaper/damselfly:dev) and refresh the metadata for the image, and let
me know if that fixes it. If not, please can you attach a sample image so I
can debug?
—
Reply to this email directly, view it on GitHub
<#507 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHANMY63KC66IYU2KFQDLX3YNMJZBAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGE4DMMZQHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Dev branch did not fix the issue, here is the file Ive been working with
janscan2024_10.png
<https://drive.google.com/file/d/163-4LpaBqEKXKly8Dg1IGwc-WxtAx8Uv/view?usp=drive_web>
…On Sun, Jan 7, 2024 at 5:43 PM Jonathan Pfeifer ***@***.***> wrote:
Sure, I'll work on it this evening
On Sun, Jan 7, 2024, 3:48 PM Mark Otway ***@***.***> wrote:
> Can you please try the dev tag of the docker image
> (webreaper/damselfly:dev) and refresh the metadata for the image, and let
> me know if that fixes it. If not, please can you attach a sample image so I
> can debug?
>
> —
> Reply to this email directly, view it on GitHub
> <#507 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AHANMY63KC66IYU2KFQDLX3YNMJZBAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGE4DMMZQHE>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
--
Jonathan Pfeifer
***@***.***
314-616-8764
|
Sorry, I confused that with an image I was testing a different program
with, this is the one that I was testing damselfly with
…On Sun, Jan 7, 2024 at 7:35 PM Jonathan Pfeifer ***@***.***> wrote:
Dev branch did not fix the issue, here is the file Ive been working with
janscan2024_10.png
<https://drive.google.com/file/d/163-4LpaBqEKXKly8Dg1IGwc-WxtAx8Uv/view?usp=drive_web>
On Sun, Jan 7, 2024 at 5:43 PM Jonathan Pfeifer ***@***.***>
wrote:
> Sure, I'll work on it this evening
>
> On Sun, Jan 7, 2024, 3:48 PM Mark Otway ***@***.***> wrote:
>
>> Can you please try the dev tag of the docker image
>> (webreaper/damselfly:dev) and refresh the metadata for the image, and let
>> me know if that fixes it. If not, please can you attach a sample image so I
>> can debug?
>>
>> —
>> Reply to this email directly, view it on GitHub
>> <#507 (comment)>,
>> or unsubscribe
>> <https://github.com/notifications/unsubscribe-auth/AHANMY63KC66IYU2KFQDLX3YNMJZBAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGE4DMMZQHE>
>> .
>> You are receiving this because you authored the thread.Message ID:
>> ***@***.***>
>>
>
--
Jonathan Pfeifer
***@***.***
314-616-8764
--
Jonathan Pfeifer
***@***.***
314-616-8764
|
You haven’t sent another image?! Can you attach it to the Github issue, or
email it here?
…On 8 Jan 2024 at 03:04:00, grainsoflight ***@***.***> wrote:
Sorry, I confused that with an image I was testing a different program
with, this is the one that I was testing damselfly with
On Sun, Jan 7, 2024 at 7:35 PM Jonathan Pfeifer ***@***.***> wrote:
> Dev branch did not fix the issue, here is the file Ive been working with
> janscan2024_10.png
> <
https://drive.google.com/file/d/163-4LpaBqEKXKly8Dg1IGwc-WxtAx8Uv/view?usp=drive_web>
>
> On Sun, Jan 7, 2024 at 5:43 PM Jonathan Pfeifer ***@***.***>
> wrote:
>
>> Sure, I'll work on it this evening
>>
>> On Sun, Jan 7, 2024, 3:48 PM Mark Otway ***@***.***> wrote:
>>
>>> Can you please try the dev tag of the docker image
>>> (webreaper/damselfly:dev) and refresh the metadata for the image, and
let
>>> me know if that fixes it. If not, please can you attach a sample image
so I
>>> can debug?
>>>
>>> —
>>> Reply to this email directly, view it on GitHub
>>> <
#507 (comment)>,
>>> or unsubscribe
>>> <
https://github.com/notifications/unsubscribe-auth/AHANMY63KC66IYU2KFQDLX3YNMJZBAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGE4DMMZQHE>
>>> .
>>> You are receiving this because you authored the thread.Message ID:
>>> ***@***.***>
>>>
>>
>
> --
> Jonathan Pfeifer
> ***@***.***
> 314-616-8764
>
--
Jonathan Pfeifer
***@***.***
314-616-8764
—
Reply to this email directly, view it on GitHub
<#507 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABFMR3SM5CFRD7OTIUNC6B3YNNO2BAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGMZDIMZVGM>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Ill attach again, its the one that starts with "fastfoto"
…On Mon, Jan 8, 2024 at 1:45 AM Mark Otway ***@***.***> wrote:
You haven’t sent another image?! Can you attach it to the Github issue, or
email it here?
On 8 Jan 2024 at 03:04:00, grainsoflight ***@***.***> wrote:
> Sorry, I confused that with an image I was testing a different program
> with, this is the one that I was testing damselfly with
>
> On Sun, Jan 7, 2024 at 7:35 PM Jonathan Pfeifer ***@***.***> wrote:
>
> > Dev branch did not fix the issue, here is the file Ive been working
with
> > janscan2024_10.png
> > <
>
https://drive.google.com/file/d/163-4LpaBqEKXKly8Dg1IGwc-WxtAx8Uv/view?usp=drive_web>
>
> >
> > On Sun, Jan 7, 2024 at 5:43 PM Jonathan Pfeifer ***@***.***>
> > wrote:
> >
> >> Sure, I'll work on it this evening
> >>
> >> On Sun, Jan 7, 2024, 3:48 PM Mark Otway ***@***.***> wrote:
> >>
> >>> Can you please try the dev tag of the docker image
> >>> (webreaper/damselfly:dev) and refresh the metadata for the image,
and
> let
> >>> me know if that fixes it. If not, please can you attach a sample
image
> so I
> >>> can debug?
> >>>
> >>> —
> >>> Reply to this email directly, view it on GitHub
> >>> <
>
#507 (comment)>,
>
> >>> or unsubscribe
> >>> <
>
https://github.com/notifications/unsubscribe-auth/AHANMY63KC66IYU2KFQDLX3YNMJZBAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGE4DMMZQHE>
>
> >>> .
> >>> You are receiving this because you authored the thread.Message ID:
> >>> ***@***.***>
> >>>
> >>
> >
> > --
> > Jonathan Pfeifer
> > ***@***.***
> > 314-616-8764
> >
>
>
> --
> Jonathan Pfeifer
> ***@***.***
> 314-616-8764
>
> —
> Reply to this email directly, view it on GitHub
> <
#507 (comment)>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/ABFMR3SM5CFRD7OTIUNC6B3YNNO2BAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGMZDIMZVGM>
> .
> You are receiving this because you commented.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub
<#507 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHANMY2YLO2U27ND5DWSG33YNOPYJAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGUYTEMZUGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Jonathan Pfeifer
***@***.***
314-616-8764
|
Don't see any attachments with that name - are you sure you actually attached it or emailed it to me? |
Thanks |
In the mean time, could you let me know what metadata fields it should be pulling Date and Caption/Description from? I have a lot of photos Im archiving and am kinda in a holding pattern until I can figure out exactly where this metadata should go, especially as there are numerous places in EXIF/IPTC that captions/descriptions in dates can go. I tried all the fields I could thing of and it didnt work, so Im sure an actual issue exits and it's not just error, but if I knew the exact fields they SHOULD be I could feel comfortable moving forward with my data entry in lightroom while you look at this without having to worry that once a solution is found I would have to go back and redo everything. |
So looking at this image, I'm seeing a few things:
Damselfly pulls in Captions and descriptions come from the IPTC directory tags I haven't debugged this with your image yet, though. |
I don't scan to jpeg for archival purposes, if I go to restore the images
later on, pngs will not contain jpeg artifacts.
…On Wed, Jan 10, 2024, 2:26 AM Mark Otway ***@***.***> wrote:
So looking at this image, I'm seeing a few things:
1. It's a PNG file. Is this intentional? Any reason why you're not
scanning to JPEG? I'll have to look and see what metadata I'm pulling for
PNG files, as it may not be the same as exif info for JPEGs.
2. Looking in Exiftool, the creation date is 1988 - is that the date
you're expecting?
Damselfly pulls in DateTimeDigitized tag for date taken. If that's not
available, I use DateTImeOriginal. And as a last resort, I use the
DateTime tag.
Captions and descriptions come from the IPTC directory tags Caption or
the Exif tag ImageDescription.
I haven't debugged this with your image yet, though.
—
Reply to this email directly, view it on GitHub
<#507 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHANMYYKIENEHPELHGQCPUTYNZGDFAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBUGM4DSMZXGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
And yes, that is the creation date I entered. If I need to enter it in
digitization date for it to correctly display, I can do that, I don't care
about the digitization date so much as the data I have for when the image
was actually captured on film.
…On Wed, Jan 10, 2024, 2:38 AM Jonathan Pfeifer ***@***.***> wrote:
I don't scan to jpeg for archival purposes, if I go to restore the images
later on, pngs will not contain jpeg artifacts.
On Wed, Jan 10, 2024, 2:26 AM Mark Otway ***@***.***> wrote:
> So looking at this image, I'm seeing a few things:
>
> 1. It's a PNG file. Is this intentional? Any reason why you're not
> scanning to JPEG? I'll have to look and see what metadata I'm pulling for
> PNG files, as it may not be the same as exif info for JPEGs.
> 2. Looking in Exiftool, the creation date is 1988 - is that the date
> you're expecting?
>
> Damselfly pulls in DateTimeDigitized tag for date taken. If that's not
> available, I use DateTImeOriginal. And as a last resort, I use the
> DateTime tag.
>
> Captions and descriptions come from the IPTC directory tags Caption or
> the Exif tag ImageDescription.
>
> I haven't debugged this with your image yet, though.
>
> —
> Reply to this email directly, view it on GitHub
> <#507 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AHANMYYKIENEHPELHGQCPUTYNZGDFAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBUGM4DSMZXGE>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
|
Okay, so it looks like the issue is that all the data you're populating is going into the XMP directory, which I wasn't really parsing much out of before (probably because most people use JPGs, and all the metadata for those usually goes into IPTC or EXIF directories. I've just pushed a change to extract the capture date from the I couldn't see any obvious caption/description data in that photo's Exif info, so it's hard to know which Photoshop XML tag to pull them from. If you can send me a sample image with the caption set to "Here is the Caption" and the description set to "Here is the description" I can add support for those too. When I get more time, I'll also see about pulling out the copyright info from the XMP tags too. But hopefully this will get you going. |
Damselfly no longer loads from the dev branch `/app/dl: cannot open shared object file: No such file or directory at Emgu.Util.Toolbox.Dlopen(String dllname, Int32 mode) at Emgu.Util.Toolbox.Dlopen(String dllname, Int32 mode) at Emgu.Util.Toolbox.Dlopen(String dllname, Int32 mode) at Emgu.Util.Toolbox.Dlopen(String dllname, Int32 mode) [15:13:58.173-0001-INF] Registered job processing source: ExifService |
From Chrome's developer tools: GET http://192.168.0.11:6363/_content/Syncfusion.Blazor.Themes/bootstrap5.css net::ERR_ABORTED 404 (Not Found) |
Weird. It's running fine for me. What OS are you running on? It might be related to some cleanup I was doing to the base image, but it's odd it works for me and not you. I'll build against the prev base image in the morning and see if that fixes it. |
Unraid. I removed the image entirely except for /appdata and redownloaded
but still have the issue.
…On Wed, Jan 10, 2024 at 4:07 PM Mark Otway ***@***.***> wrote:
Weird. It's running fine for me. What OS are you running on?
It might be related to some cleanup I was doing to the base image, but
it's odd it works for me and not you. I'll build against the prev base
image in the morning and see if that fixes it.
—
Reply to this email directly, view it on GitHub
<#507 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHANMY6BCC7SA4HYQGF7COLYN4GJDAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBVHAYTKMJQGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Jonathan Pfeifer
***@***.***
314-616-8764
|
I'll try the old base image tomorrow. Might have optimised it too much.
…On Wed, 10 Jan 2024, 22:09 grainsoflight, ***@***.***> wrote:
Unraid. I removed the image entirely except for /appdata and redownloaded
but still have the issue.
On Wed, Jan 10, 2024 at 4:07 PM Mark Otway ***@***.***> wrote:
> Weird. It's running fine for me. What OS are you running on?
>
> It might be related to some cleanup I was doing to the base image, but
> it's odd it works for me and not you. I'll build against the prev base
> image in the morning and see if that fixes it.
>
> —
> Reply to this email directly, view it on GitHub
> <
#507 (comment)>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/AHANMY6BCC7SA4HYQGF7COLYN4GJDAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBVHAYTKMJQGQ>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
--
Jonathan Pfeifer
***@***.***
314-616-8764
—
Reply to this email directly, view it on GitHub
<#507 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABFMR3T3V7OIF2XHFXTK32LYN4GSDAVCNFSM6AAAAABBQTJ7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBVHAZDCMBXGM>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Can you try pulling |
Its working now, date is showing up and faces areas are tagged which was unexpected but awesome. This copy of the image im adding has the caption field filled out, for all intents and purposes I only use the "caption" field and not an additional description field. Also, is there a way in the UI to turn the face region overlay off? I love it as it makes it really easyl to identifiy whos who, but it would be great if it could be quickly hidden to have a clearer view of the picture. |
It should be pulling in caption, if it's not, I'll have a look. If you click the "show objects" icon that should turn off face rects, and other AI labels. |
Cool, found the show objects button. Yeah, I tested sever different fields that were some version of caption/description and it wouldnt pull from any. The file I just loaded has the "This is the caption" value in the field that makes the most sense in lightroom. |
Also, unrelated and not really causing me an issue, but "sharpening..." never seems to go away above the image on the screen, just a heads up |
I've just downloaded the image above, but even if I run it through exiftool I can't see any metadata in the image with the word "caption". Are you sure you a) saved the caption, and b) uploaded the right image? Alternatively, if you could just run Unsure about the sharpening display. What's supposed to happen is that it loads a low-res image immediately, and then loads a higher-res preview, but depending on the system that can sometimes take a second or so to be generated. Once it is, it's loaded, and the 'sharpening' message should go away. It's possible that your system is so fast some race condition is happening? :) |
Yes, I just spotted that. Must be having a bad morning. So it seems the library I use to extra metadata tags can't load the XMP block of data from that image. I get an error returned saying it's an unsupported encoding type. Don't know why. Do you mind if I raise an issue and attach the photo so the developer can see if there's a fix? In the meantime you might need to see if there's some other way you can do it. Perhaps you can also save the XMP stuff as a sidecar file? It's possible I could rewrite the code to use exiftool, but that would be a significant effort, so wouldn't happen anytime soon (and I'd like to avoid doing it if possible). |
Using the image is fine with me. I actually explored the sidecar option but lightroom doesnt seen to be able to do that for files that arent RAW. Ive already spent a couple years on this project so a little more time while the developer works on it wont hurt, Ill just work on the other facets of it first and leave this chunk of photos for last. |
Okay, great. With a bit of luck we'll get a fix (the developer is quite responsive) and then you can just rescan, but hopefully the changes made this week will at least get you going! |
Raised the issue here: drewnoakes/metadata-extractor-dotnet#356 |
🤷🏻♂️ |
@Webreaper It does look like we have a release candidate for the metadata library if you care to look at it now, or if you'd rather wait for the release thats fine |
Yep, I need to sort out a release. But there's some other bits that are half done and aren't ready for release, but need tidying or hiding behind a feature flag before I push it out. Will try and do that in the next week or so. In the meantime, you can continue using dev... |
Just to clarify, I meant that @drewnoakes had a RC for metadata-extractor-dotnet. I wasn't sure from your response if we were talking about the same thing or not |
And your dev seems to be working fine ever since the sharpening thing worked itself out |
The 2.9 release has a bunch of performance gains and some corresponding API changes (though most users won't notice them). Let me know if you encounter any issues updating to the RC build. We have a few more changes coming before we'll release the final build. |
Sorry, missed what we were talking about. D'oh. I'll update to the new RC and see how it looks. |
Updated to 2.9 rc1, and pushed - new dev image should be there soon for some testing. |
Will keep an eye out |
Update, Still no information in the description field, there seems to be an error that may pertain to that in the logs when I reindex 02:27:51.470 | INF | ExtraLarge thumb created for fastfoto_0804291.png [load: 53ms, scale: 3ms, save: 41ms -- | -- | -- 02:27:49.492 | INF | Search: 0 images found in search query within 4ms 02:27:44.965 | INF | Processing AI image detection for fastfoto_0804291.png... 02:27:44.587 | INF | Read metadata for /pictures/Family Png 7/fastfoto_0804291.png (ID: 1673) | | | | 02:27:44.586 | ERR | Exception while parsing XMP face/region data: System.Collections.Generic.KeyNotFoundException: The given key 'exif:Description' was not present in the dictionary. 02:27:44.355 | INF | Full index compelete. 02:27:43.758 | INF | Performing full index. |
Will take a look when I can. |
I also just now noticed that I no longer see 3 metadata fields, including the caption field, in the Image properties section at all, empty or otherwise, after a refresh |
I haven't tested the build, I literally just updated the dependency. I'll take a look when I get a chance. |
Its cool, just wanted to let you know that I noticed those fields missing |
While it didnt resolve the error in the logs or populate the fields, restarting the whole docker container did bring back the metadata fields that had vanished, so that appears to have just been another fluke |
This error message no longer appears in most recent dev image, however original problem persists |
@Webreaper Were you able to get this working on your end? The fields are still not populating on mine |
Sorry, couple of oversights... pushed a fix to dev - can you confirm it works, and if so I'll push a new release. |
Did you refresh and rescan metadata? It's definitely working for me now. To sanity check, rename the file so it's ingested from new. |
Im getting a persistent error with metadata processing An exception occurred in the database while saving changes for context type 'Damselfly.Core.Database.ImageContext'. -- | | | | | | 16:42:07.876 | ERR | Failed executing DbCommand (0ms) [Parameters=[@p23='?' (DbType = Int32), @p0='?' (DbType = DateTime), @p1='?' (DbType = Double), @p2='?' (Size = 7), @p3='?' (DbType = Int32), @P4='?', @p5='?', @p6='?', @P7='?' (DbType = DateTime), @p8='?' (Size = 19), @p9='?', @p10='?', @p11='?', @p12='?' (DbType = Boolean), @P13='?' (DbType = Int32), @p14='?', @P15='?' (DbType = Int32), @p16='?' (DbType = DateTime), @p17='?' (DbType = Double), @p18='?' (DbType = Int32), @p19='?' (DbType = Double), @p20='?' (DbType = Int32), @p21='?' (DbType = DateTime), @p22='?' (DbType = Int32) |
Is there more error info than that in the logs? That's not much to go on. I wonder if the DB schema change didn't work when it upgraded to the new version.... If the metadata isn't working, you're not going to see the new data. |
Actually, ignore this - I can see the error. Investigating. |
Okay, dev should be better now. Can you try it out? |
SUCCESS! |
WOOT!!! Thanks. I'm going to make a patch release. :) |
Im running into an issue where the "Date Taken" information seems to be displaying the date the image was imported into damselfly. I am editing the capture date/time metadata in lightroom, though when I write it to the file, it does not change in damselfly. Other metadata, such as tags, do change when updated. Additionally, the date shown in damselfly does not seem to match any data in the metadata, as both the existing capture and digitization dates were in 2022 and the changed date is 1988, though damselfly has always shown 2023. No date ending in 2023 has ever existed in the file metadata.
The text was updated successfully, but these errors were encountered: