-
Notifications
You must be signed in to change notification settings - Fork 103
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
Jellyfin uses this project but some subtitles don't display #80
Comments
What if you remove just |
It doesn't work. But if a delete \fad(...) in first picture, it works. |
this is an issue with the font file, subtitle octopus doesnt have a good font fallback and the file you played it back on probably didint include an embedded font file at all, it's an issue with the file and not jellyfin/S.O., most likely, it could also be an issue with the chars themselves, the file itself would be nice since i cant find the release anywhere edit: managed to find the release, seems like the subtitles are embedded in a very weird way and neither vlc or any web player picks them up |
@swhoro also note that we have a fork at JustAMan/JavascriptSubtitlesOctopus due to some changes that never got merged here, so you'd have to make the change there. We technically have an "official" fork at jellyfin/JavascriptSubtitlesOctopus but I think the former is more recent, so we should probably migrate everything into one repository at some point. |
both of those repos are private? |
I fixed the links, I haven't looked at either in months. It looks like the official one is most recent, so any pull requests should be directed here first, and then to our fork if the maintainer decides they aren't within scope. |
oh, my bad, but this is a file issue, not a S.O. issue, but yeah, this repo is more up to date because they introduced "an api" but seems like its broken anyways |
jellyfin/JavascriptSubtitlesOctopus is what we use as a dependency. It's basically this repository with some rendering speed improvements (A feature we never upstreamed because the person who made it is less active these days) and without the added API (We don't need it so we never bothered to bring in the changes) |
So are my font files broken? |
I have tried many videos and ass recently, it seems all subtitles with {/fad(...)} will not show up in jellyfin. Unfortunately, I have deleted videos and ass files in second picture... |
|
Thanks for your reply! |
Yes, you need fonts for all glyphs. For complex typeset ASS you even need to have the exact same font available as the subtitle-author to get the intended result.
Jellyfin uses a fork of JSO with custom patches (but and older libass). Perhaps they use different versions for the web and iOS-Frontends, or maybe the iOS-App is using libass directly without JSO as a middle layer? |
We'd like to upstream these at some point, but the team member who made them went MIA about a year ago and I'm not personally sure what is and what wasn't upstreamed (And I'm not well-versed in web assembly development). Iirc, our patches were mostly around making rendering faster for more complex animations. I do want to use the upstream version as-is in our Vue client at some point though, so I might look into bringing them over, if #106 doesn't do so before. |
this is 100% a jellyfin/font issue, I tested this aggressively when making my own version of JSO, and all glyphs worked no problem as long as the font supported them, all animations I've encountered from fansubs also seem to work |
about those changes of yours, they only seemed to benefit very low-end devices without hardware acceleration, and pre-rendering didn't work that well for files with >1k subtitles |
Good to hear there's interest in (helping to) upstream the patches, not so good to hear about the MIA-part; I hope the missing member is doing well. |
https://1drv.ms/f/s!AmlVPwgDqKkuvjvPlDyskaUkZkOG |
@swhoro At which line or time does the issue occur for you? |
At line 362, when Japanese character in op show up |
I can reproduce the issue with some glyphs remaining tofu, but only with the WOFF2-font; using the TTF version everything renders correctly (in both upstream and jellyfin JSO). It appears something went wrong when you converted the TTF to WOFF2; you can use the orignal TTF instead. I cannot reproduce any animation issues in neither upstream nor jellyfin's JSO-builds using the default renderer. It may be that you're CPU (single-thread) just isn't fast enough to execute the WASM in time inside the browser, but it (barely?) works out with the iOS-app. If you/jellyfin is using the If you think there's more to it we can reopen the issue. For now it appears to me like this is no issue with JSO or jellyfin. |
For example, in this picture, from line 27 to 47, text starts with {\fad(200,300)\be1} or {\fad(500,500)\be1\an8} will not be loaded in the video.
If I delete all the text in {...} and {}, those subtitles can be loaded correctly.
However, line 48 can display.
I have another ass file:

These subtitles with {...} can be displayed correctly.
Here is the issue in jellyfin:jellyfin/jellyfin-web#2081
The text was updated successfully, but these errors were encountered: