Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upGitHub is where the world builds software
Millions of developers and companies build, ship, and maintain their software on GitHub — the largest and most advanced development platform in the world.
[twitch] Embedding subs (chat) failing #25366
Comments
|
Because Twitch subs are in a format that can't be embedded. |
|
Oh, okay. I thought this was working at some point, must be mistaken. Is there any other way to shove them in the same file, like is done with youtube metadata? Anyway feel free to close, although I do think it would be a plus if there was a better error message for unsupported operations like this. |
You can close the issue since you opened it. Just click the button. To answer your first question, you'd have to manually transcode the JSON subs format to SRT, then embed it into the file, but that won't look pretty like you think it will and I don't even think you can transcode from JSON to SRT. |
|
On 2020-06-06 20:26, Paul Henning wrote:
> Anyway feel free to close, although I do think it would be a plus if
> there was a better error message for unsupported operations like
> this.
You can close the issue since you opened it. Just click the button.
I don't care to close this because I think there's still an action item
(print a better error message).
One contributor should please take charge and decide whether that should
be done (to avoid lack of responsibility), and close it as wontfix if
not :).
> Oh, okay. I thought this was working at some point, must be
> mistaken.
>
> Is there any other way to shove them in the same file, like is done
> with youtube metadata?
>
To answer your first question, you'd have to manually transcode the
JSON subs format to SRT, then embed it into the file, but that won't
look pretty like you think it will and I don't even think you can
transcode to that format.
By "shove them in the same file", I meant that for youtube,
--add-metadata adds youtube's JSON metadata into the .mkv somehow after
download (I assume as a metadata track, although it looks like there
might be an xattr option? I really need to learn more about how this
works). Sorry for being a bit vague the first time around. It seems like
that would be useful here, even if actual subtitle embedding isn't
possible. However this is a poorly-thought-out idea, and definitely a
separate issue, I was just thinking aloud.
|
|
JSON metadata is different from subtitles, they don't even describe the same things. |
|
Yes. I was suggesting to use the same mechanism to include the 'XXXrechat.json' as a track as is used to include the youtube metadata JSON as a track. And I'm retracting my suggestion, in any case. Feel free to add a check for the incompatible: twitch, --all-subs, and --embed-subs with a nice error message. I feel like this conversation isn't going anywhere productive so I'm closing this. |
A bit of background: Twitch has "Video on demand"=VOD, basically whole archived streams, and "clips", 30-second segments. VOD has chat, I think clips don't. Since VOD disappears after a month, I archive it--but also you may need to find a new sample URL to test on depending on how long it takes to get to this, please don't just close the bug because the URL is expired.
I did survey existing bugs. This may be related to #18593, I can't tell if that issue was fixed and just not cleaned up, but this is a new error message so I'm filing it as a new bug. I didn't see any other bugs that have this error message and the word "twitch", open or closed.
Verbose log
Description
As best I can tell, the flags
--all-subs --embed-subscauses videos with chat to fail download on twitch.Edit: Oh also, I believe the expected behavior on twitch is that "subs" should some version of twitch chat, embedded as a subtitle track, which isn't super obvious :)
Thanks everyone for maintaining this great program.