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

Added full list of media files (and associated data) from the HEADER section #39

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

JaviMaciasG
Copy link

The header of the ELAN eaf files can contain multiple media files, so that I made a simple modification to read them all. It is implemented as a list of dictionaries contanining the data associated to each media file. It follows the implementation of the properties that are already read.

I kept the speach functionality as it was, to avoid possible incompatibilities and to keep things simple. I guess that the setter and getter methods related to the media_file, media_url, mime_type, relative_media_url should be generalized to allow for setting multiple media descriptors, but this can be done later.

The header of the ELAN eaf files can contain multiple media files. This
is implemented as a simple list of dictionaries contanining the data
associated to each media file.
@erenyenigul
Copy link

This looks perfect! Just what I needed! Please merge this to the main.

@JaviMaciasG
Copy link
Author

This looks perfect! Just what I needed! Please merge this to the main.

I don't know if the repo owner is still active as his last commits in GitHub are dated 2022. Let's hope he can find the time to do the merge.

The idea is allowing the generation of releases to be used internally in
our group
The idea in the future is fixing the @Property and @*.setter related to
media content. I did not fix it right now as this is not compulsory for
our initial intended use, but should be doing to provide a consistent
functionality with media files.
Intended for future use, to allow relative time handling. In our group
we are using elan files from different sources that might not be
synchronized. time_origin is a way to unify the time references.

I did not further implement it (TODO)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants