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

Add better support for video #12

Open
realies opened this issue Feb 7, 2018 · 4 comments
Open

Add better support for video #12

realies opened this issue Feb 7, 2018 · 4 comments
Assignees

Comments

@realies
Copy link

realies commented Feb 7, 2018

Noticed that the displayed metadata for video files from cameras such as Canon 5D and 7D is less than it could be. Would like to see the camera details, codec types and bit rates for all media streams.

screen shot 2018-02-08 at 00 57 44

@gino0631
Copy link
Owner

gino0631 commented Feb 8, 2018

OK, thanks for reporting this. Would it be possible for you to attach a small sample video?

@gino0631 gino0631 self-assigned this Feb 8, 2018
@realies
Copy link
Author

realies commented Feb 8, 2018

Sure, OO4A3570.MOV.zip

@gino0631
Copy link
Owner

Thanks for the sample, I have made some improvements.
Camera details (as well as author and copyright strings) are OK.
The name of video codec is not returned by the getID3 library for some reason, but FourCC code is present, so I can do the mapping in the app (though I don't like the idea to have mappings for all possible FourCC codes). As for the bit rates, only an overall bit rate is returned, so I've opened an issue in getID3 project to see what can be done.

@tzxmaster
Copy link

Hi ! Since this issue is still open, I'll mention another information that could be added hopefully without much effort. That would be the creation time of the videos. It is available for photos and there are some details available in exiftool (not sure which would be best since for my videos, all are the same). I ask because I build a gallery app that sorts media based on exif data, it uses this application as a dependency and currently I use the mtime column from the database for sorting videos but this feature might be useful for all users

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants