search

zzag / plasma5-wallpapers-dynamic

Dynamic wallpaper plugin for KDE Plasma
328 stars 18 forks source link

kdynamicwallpaperbuilder: GPS Data not recognized #136

Closed OWL4C closed 8 months ago

OWL4C commented 8 months ago

When using "*" for the Solar Metadata kdynamicwallpaperbuilder returns Failed to compute the position of the Sun based on GPS coordinates and the time when the photo was taken. Please check that the specified image actually has GPS coordinates in its Exif metadata. You can do that with a tool such as exiftool. I have this error with both my actual dataset as well as a test file, which is attached. In case the exif gets stripped by github, i have set -GPSLongitudeRef=E -GPSLatitudeRef=N -GPSLatitude=0.0 -GPSLongitude=0.0 manually with exiftool and it shows working when looking at the file with exiftool again, as well as the normal File Explorer Exif Info. chinese-reed-8493547_1280

The picture is from https://pixabay.com/photos/chinese-reed-reed-grass-plant-8493547/

zzag commented 8 months ago

Hmm it looks like DateTimeOriginal or GPSTimeStamp+GPSDateStamp are missing. They are needed to figure out when the picture was taken to compute the position of the Sun. Both location and time are needed for that

zzag commented 8 months ago 
File Modification Date/Time     : 2024:01:15 17:17:52+02:00
File Access Date/Time           : 2024:01:15 17:18:44+02:00
File Inode Change Date/Time     : 2024:01:15 17:17:52+02:00

these are not suitable because they would produce incorrect results

zzag commented 8 months ago

I will improve the error message. Is it possible to get exiftool dump for one of your images (please redact sensitive information such as location)?

OWL4C commented 8 months ago

Ok, thinking about it makes it seem pretty stupid. I definitely miss the GPS Timestamp tags since i added the Location Tags manually, since my camera doesn't have the functionality It would be really good to have more info about which tags are required in either/and the error or the README. Also maybe you could add some more info to the error, print out all tags required, if they exist, and which part (calculation or reading) errored?

zzag commented 8 months ago

It would be really good to have more info

Yeah, I'll update the error messages and update README. Thanks!