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

feat: add subtitle support #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

poontology
Copy link

Tested with latest Heresphere (0.10.2) and DeoVR (13.19.1687) on Quest 2.

To add subtitles in stash name the subtitle file same as the video file except with .srt extension or .en.srt, .de.srt etc for different languages, then rescan stash library so it finds the file, if there's no language code in the filename it shows up as "00" in the API.

There's btw some idiosyncrasies relating to subtitles in stash and the players:

stash: /caption API reformats all subtitle files into webvtt which looks almost the same as subrip/.srt but if you test samba shared subtitle vs subtitle over API it might work differently because of that reformatting (there was also a "bug" in previous heresphere version that didn't render subtitles because the millisecond separator is different between those formats)

heresphere: doesn't have a language selection so if stash has multiple subtitles for one video it picks the first one in the json array which appear to be alphabetically sorted by the language code. when heresphere detects that video has a subtitle the CC icon next to timeline turns purple but I found out that if the Name property in stash-vr json reply doesn't end with ".srt" it refused to render subtitles (probably because it uses it to decide between different subtitle parsers).

deovr: also picks the alphabetically first subtitle although it does appear to have a dropdown in the video menu to change the active subtitle it didn't seem to work for me but I think most users only really care about having one subtitle so not sure if it's even worth reporting/investigating.

Resolves #37

@GoldstarZV
Copy link

thank you for this! looking forward to testing it

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.

Subtitles on heresphere?
2 participants