Skip to content
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

Edit metadata on videos #105

Closed
Donkey-Doug opened this issue Oct 12, 2021 · 9 comments
Closed

Edit metadata on videos #105

Donkey-Doug opened this issue Oct 12, 2021 · 9 comments
Labels
edition metadata P2 random bugs & feature requests type:feature New feature or request video

Comments

@Donkey-Doug
Copy link

Editing the date and removing metadata does not work on videos. The buttons are greyed out.

Screenshot_20211012-210131_Aves

@deckerst
Copy link
Owner

deckerst commented Oct 13, 2021

Indeed, some features are limited to a few formats.

For now and the foreseeable future:

  • editing the date is limited to JPEG, PNG, WEBP, DNG,
  • removing metadata is limited to JPEG, TIFF.

edit: details on editing support by format on the wiki

@deckerst deckerst added metadata P3 valid issue to be addressed in the future video labels Oct 13, 2021
@deckerst deckerst changed the title Edit date & remove metadata on videos Edit metadata on videos Nov 16, 2021
@deckerst deckerst added the type:feature New feature or request label Nov 21, 2021
@Ingegneus
Copy link

might it be possible to read/write the metadata to existing sidecar files? it would be nicer to have the metadata directly in the file, but this might be temporary solution.

@deckerst
Copy link
Owner

Fair suggestion, but I won't support sidecar files. Among other reasons is a technical one: sidecar files are not media content, so it's out of scope, as Aves is not a file manager.

But here, the real issue is that I'm dragging my feet to implement MP4 metadata edition. I should tackle it, but I got carried away with more interesting stuff instead!

@kha84
Copy link
Contributor

kha84 commented Oct 2, 2022

I spent a quite time analyzing all existing open source media managers - https://orange-pi-4-lts.blogspot.com/2022/09/ultimate-guide-to-linux-photo-media.html?m=1 - and even Digikam with all its recent move towards exiftool doesn't fully support that. If you implement this, you'll be the first hero ever.

@deckerst
Copy link
Owner

deckerst commented Oct 2, 2022

@kha84 nice write-up. That kind of summary is always useful.

if you had any kind of GPS tracker with you, which was writing a .gpx track file.

Is that a use case for you? I thought about supporting this when i worked on location editing, but i personally don't need it and there was no explicit demand at the time.

As for videos, I've done some research and it seems writing metadata to MP4 could be done with simple libraries, but for other video formats only the big guns like exiftool or ffmpeg can handle them, and those heavy libraries are not an option for Aves.

@deckerst deckerst added P2 random bugs & feature requests and removed P3 valid issue to be addressed in the future labels Oct 2, 2022
@kha84
Copy link
Contributor

kha84 commented Oct 2, 2022

Reg .gpx tracks - nopes I don't need that. I just put that to my post (it's still unfinished) to be a bit more fair to a guy who developed that software. Because it's definitely has a usecase. But just not for me.

From features perspective, my own requirements are fullfilled by Aves at 90%. What I'm missing is:

a) a way to put metadata to my mp4 videos I'm occasionally shooting with my phone. This is the most wanted thing.

b) some kind of OpenCV thing to recognize faces on photos. This is highly desirable, but can wait.

If that will be done, Aves becomes just a golden standard of how these kind of apps should be made like. A digikam in your pocket

@xmha97
Copy link

xmha97 commented Oct 9, 2022

Set GPS location to videos using exiftool

exiftool -GPSAltitude="0" -UserData:GPSCoordinates="34.6363, 50.874900000000004, 0" -Rotation="0" -XMP:GPSLatitude="34.6363" -XMP:GPSLongitude="50.874900000000004" -GPSLongitudeRef="West" -GPSLatitudeRef="North" VID_20221006_094430.mp4

https://www.google.com/maps/@34.6363,50.874900000000004,21z
cc: @deckerst

@deckerst
Copy link
Owner

@xmha97 see #230

deckerst added a commit that referenced this issue Oct 18, 2022
@deckerst
Copy link
Owner

MP4 only, see wiki for support details.

deckerst added a commit that referenced this issue Oct 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
edition metadata P2 random bugs & feature requests type:feature New feature or request video
Projects
None yet
Development

No branches or pull requests

5 participants