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

Empty segmentTemplate when subtitle file is empty #422

Closed
jakubvojacek opened this issue Jun 30, 2018 · 5 comments
Closed

Empty segmentTemplate when subtitle file is empty #422

jakubvojacek opened this issue Jun 30, 2018 · 5 comments
Labels
status: archived Archived and locked; will not be updated type: external An issue with an external dependency; not our issue; sometimes kept open for tracking

Comments

@jakubvojacek
Copy link

System info

Operating System: latest debian
Shaka Packager Version: version 1600909-release (master)

Issue and steps to reproduce the problem

Packaging VOD (actually a catchup event, but thats like VOD) with subtitles. But there happened to be no subtitles available in the event's timeframe. Therefore the vtt file was empty. The content was packaged successfully but the generated MPD is not playable via shaka player (plays with dashjs though) due to SegmentTemplate does not contain any segment information: the SegmentTemplate must contain either an index URL template a SegmentTimeline, or a segment duration. I am attaching sample MPD for reference 2018-06-30-17-29-412343-euromaxx.mpd.txt.

Is this an issue in shaka packager or shaka player? One possible workaround would be to check the generated vtt file before packaging and remove the subtitle portion of the command but would like to avoid that.

Packager Command:

packager  'in=/data/assets/channels//ct24-9/recordings/2018/06/30/2018-06-30-18-00-412345-zpravy/output.mp4,stream=0,init_segment=2018-06-30-18-00-412345-zpravy_video_640_360.mp4,segment_template=video_640_360/video_640_360_2018-06-30-18-00-412345-zpravy-$Number$.mp4' 'in=/data/assets/channels//ct24-9/recordings/2018/06/30/2018-06-30-18-00-412345-zpravy/output.mp4,stream=1,init_segment=2018-06-30-18-00-412345-zpravy_video_720_410.mp4,segment_template=video_720_410/video_720_410_2018-06-30-18-00-412345-zpravy-$Number$.mp4' 'in=/data/assets/channels//ct24-9/recordings/2018/06/30/2018-06-30-18-00-412345-zpravy/output.mp4,stream=2,language=cs,init_segment=2018-06-30-18-00-412345-zpravy_audio_cs.mp4,segment_template=audio/audio_cs_2018-06-30-18-00-412345-zpravy-$Number$.mp4' 'in=/data/assets/channels//ct24-9/recordings/2018/06/30/2018-06-30-18-00-412345-zpravy/output.vtt,stream=text,init_segment=2018-06-30-18-00-412345-zpravy_subtitle_init.mp4,segment_template=subtitle/ttv_$Number$.m4s,language=cs' --mpd_output 2018-06-30-18-00-412345-zpravy.mpd --generate_static_mpd --hls_master_playlist_output 2018-06-30-18-00-412345-zpravy.m3u8 --fragment_duration 3.2 --segment_duration 3.2  --suggested_presentation_delay 5 --min_buffer_time 3.2 --minimum_update_period 5 --default_language=cs
@kqyang
Copy link
Collaborator

kqyang commented Jul 2, 2018

@jakubvojacek This is similar to this feature request #354. We are hesitant to support things like this directly in Shaka Packager as we think it is better to be handled outside of packager. We think it is better for usability and clarity with the assumption that If the stream is provided to Shaka Packager, then it should be a valid stream and it should be present in the generated manifest. Of course, we will reconsider the decision if there are strong user requests to support this.

@joeyparrish Joey, do you think if it makes sense to support a manifest like this in Shaka Player?

@jakubvojacek
Copy link
Author

We think it is better for usability and clarity with the assumption that If the stream is provided to Shaka Packager, then it should be a valid stream

But the file was valid, I believe, even though empty, still valid.

When you're watching a live channel with closed captions available, you have the option to switch them off or on. When you're watching a recording of some event from the same channel, the captions should still appear as available (even when empty for given time period), that is the correct user experience in our opinion, that is why I am would like to avoid not having the subtitles generated.

@joeyparrish
Copy link
Member

This is something that Shaka Player should be able to handle. I'll file an issue there.

@joeyparrish
Copy link
Member

@kqyang
Copy link
Collaborator

kqyang commented Jul 2, 2018

Thanks Joey!

@jakubvojacek Your argument makes sense to me. I'll close the bug and leave it to be handled by Shaka Player. Feel free to re-open the bug or post comments if you think it is useful to handle it in packager as well.

@kqyang kqyang closed this as completed Jul 2, 2018
@kqyang kqyang added type: external An issue with an external dependency; not our issue; sometimes kept open for tracking and removed needs triage labels Jul 2, 2018
shaka-bot pushed a commit to shaka-project/shaka-player that referenced this issue Jul 3, 2018
See also shaka-project/shaka-packager#422

Closes #1484

Change-Id: I293468a39918a4a8487d15ec70b365eec3dd1ebc
joeyparrish added a commit to shaka-project/shaka-player that referenced this issue Aug 6, 2018
See also shaka-project/shaka-packager#422

Closes #1484

Change-Id: I293468a39918a4a8487d15ec70b365eec3dd1ebc
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Aug 31, 2018
@shaka-project shaka-project locked and limited conversation to collaborators Aug 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated type: external An issue with an external dependency; not our issue; sometimes kept open for tracking
Projects
None yet
Development

No branches or pull requests

4 participants