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

[Feature] Subtitles support and audio snippets in Test from media #156

Closed
wants to merge 34 commits into from

Conversation

ProgramComputer
Copy link

@ProgramComputer ProgramComputer commented Oct 7, 2023

The capability to parse subtitle files was added. Add a vtt, srt ... as a text, and link the corresponding audio-containing media. The audio will snip to fit the sentences in test mode, so you can play the tracks while testing - a play button will appear if possible.

As implied, it is not supported in long text mode.

A minor change was the Read Words Aloud checkbox in the test didn't keep the check selection when requests were made, so voice synthesis should now work in test mode as well.

I tested it with a video file, so any media file with audio supported on the browser will work.

SeStartSec and SeEndSec were added to Sentences table. These are in milliseconds. How the milliseconds are played depend on the browser. These may need to be adjusted by browser

@HugoFara HugoFara added enhancement Develop an existing feature new-feature A new feature labels Dec 27, 2023
@ProgramComputer ProgramComputer force-pushed the subtitles branch 3 times, most recently from 9a2ca1e to 0a9eaf5 Compare December 27, 2023 16:32
@ProgramComputer ProgramComputer marked this pull request as draft January 3, 2024 17:22
@HugoFara
Copy link
Owner

HugoFara commented Jan 5, 2024

Actually it's a feature I thought about, and your solution seems to be great. Yet I think it will be difficult to integrate it into LWT elegantly, as you would be able to create a text from an audio file, and it implies many different changes (on top of the ones you already brought).

As I also want to rebuild the media feature (into something HTML5) compatible, it will be great to integrate your PR at that time, see that for the future!

@HugoFara HugoFara closed this Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Develop an existing feature new-feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants