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

Silence detection: Skip attaching smil file if empty #5416

Merged
merged 1 commit into from Nov 22, 2023

Conversation

geichelberger
Copy link
Contributor

@geichelberger geichelberger commented Nov 7, 2023

Under certain circumstances, for example, if the non-silent segments were smaller than the configured minimum voice length, the SMIL file contained no segments.

This patch changes the behavior so the smil file won't be attached to the media package if it contains no segments.

@geichelberger geichelberger changed the title Silence detection return at least one segment Silence detection: Return at least one segment Nov 7, 2023
@geichelberger geichelberger marked this pull request as draft November 7, 2023 16:42
Under certain circumstances, for example, if the non-silent segments
were smaller than the configured minimum voice length, the SMIL file
contained no segments.

This patch changes the behavior so the smil file won't be attached to
the media package if it contains no segments.

Signed-off-by: Gregor Eichelberger <gregor.eichelberger@tuwien.ac.at>
@geichelberger geichelberger changed the title Silence detection: Return at least one segment Silence detection: Skip attaching smil file if empty Nov 14, 2023
@geichelberger geichelberger marked this pull request as ready for review November 14, 2023 14:24
@gregorydlogan gregorydlogan self-assigned this Nov 22, 2023
@gregorydlogan gregorydlogan merged commit 99cb6f9 into opencast:r/14.x Nov 22, 2023
3 checks passed
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.

None yet

2 participants