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

[Bug]: auto select external subtitle #1162

Open
mitz1 opened this issue Mar 22, 2024 · 17 comments
Open

[Bug]: auto select external subtitle #1162

mitz1 opened this issue Mar 22, 2024 · 17 comments

Comments

@mitz1
Copy link

mitz1 commented Mar 22, 2024

@courville want to report that something went wrong and after [v6.2.63 release] they don't even load tvshow-s01e01.fr.srt nor a tvshow-s01e01.srt but an English one. I reverted to a [v6.2.63 release] . ( Tested with v6.2.64 release and v6.2.65 )
This is my previous comment: Now I understand the logic that the program first looks for built-in subtitles for the language we have chosen by default. In your case French. After not finding it, it chooses external subtitles tvshow-s01e01.fr.srt. But it would be nice (and I think more logical) if it can't find the tvshow-s01e01.fr.srt file, then look for tvshow-s01e01.srt.

Originally posted by @mitz1 in #897 (comment)

@mitz1 mitz1 changed the title auto select external subtitle [Bug]: auto select external subtitle Mar 22, 2024
@nonelocker
Copy link

nonelocker commented Apr 3, 2024

And there are malfunction that is auto selection of internal subtitles after 6.2.60. (6.2.59 is good well)

@mitz1
Copy link
Author

mitz1 commented Apr 16, 2024

In fact, I've tested both this version and v6.2.63 quite extensively and it turns out that they work equally well for me and in most cases catch external subtitles even with the same names without using the .fr.srt advice from @courville (with very few exceptions). And in fact, it is best to rename the subtitles to fr.srt on his advice so that there are no surprises. After version 6.2.63 the versions are a disaster and maybe on my big mouth. It's strange that I had a problem with this version earlier, but this time I found and deleted a .db file in the root I think (I think the name was files.db, but I'm not sure, and I don't know if it has anything to do with this app) and deleting before each test all hidden files with the extension . xml (which turns out to remember last selected subtitles - even wrong ones from newer versions). Also clear Nova video player cache and data before each test. As I said before, it will be for the best that the program first looks for built-in subtitles for the language we have chosen by default and if it can't find them to choose external subtitles tvshow-s01e01.fr.srt. But it would be nice (and I think more logical) if it can't find the tvshow-s01e01.fr.srt file, then look for tvshow-s01e01.srt. The worst thing is that the latest version v6.2.67 release is updated in google play and I will have to stop the automatic updates I hope that in future versions the people working on this nice project will solve the problems. #897

@mitz1
Copy link
Author

mitz1 commented May 13, 2024

I stay with version 6.2.63. Tested v6.2.73 release

@courville
Copy link
Contributor

@mitz1, when I have video.mkv and video.srt, video.en.srt, video.fr.srt and I select preferred language french on nova, it does pick correctly the external sub if there is no french sub in the mkv.
Please let me know if it is the case for you (changing the language to yours of course).

@HosePL
Copy link

HosePL commented May 19, 2024

And there are malfunction that is auto selection of internal subtitles after 6.2.60. (6.2.59 is good well)

Same here.

It doesn't matter how external subtitle file is named, player always choose english internal subtitles, despite polish ones in settings.

If there is no external subtitle file, the player always choose embedded english subtitles even if there are other embed subtitles in preferred / configured language.

Reinstalled the app, cleared cache, nothing helps.
The same issue on Sony's Google TV and Sharp's Android TV.

@courville
Copy link
Contributor

courville commented May 19, 2024

@HosePL, thanks for the feedback. I have located an error in my code indeed.

courville added a commit to nova-video-player/aos-Video that referenced this issue May 19, 2024
courville added a commit to nova-video-player/aos-Video that referenced this issue May 19, 2024
…language track in video

When no sub matching preferred language found set the first track but update it in mVideoInfo

See: nova-video-player/aos-AVP#1162
@courville
Copy link
Contributor

@mitz1 and @HosePL please try https://github.com/nova-video-player/aos-AVP/releases/tag/v6.2.75 if this is any better on your side.

@HosePL
Copy link

HosePL commented May 20, 2024

Just tested 6.2.75 and it works correct with both embed and external subtitles. Thanks @courville.

@HosePL
Copy link

HosePL commented May 25, 2024

@courville I'm afraid there is still some issue.
My subtitles language is set to polish. Nova choosed correctly polish from embed or external (xxx-pl.srt) subtitles, but displays english subtitles.
To get right subtitles I have to change subtitle language manualy to any other language and change it back to polish.
IMG_0663

@courville
Copy link
Contributor

courville commented May 25, 2024

@HosePL is this with latest release 6.2.76 (I have refactored the overall logic to simplify and avoid code duplication)?

Do you confirm that this is with a video that has only internal subtitles or you have a mix (important to reproduce)?

@HosePL
Copy link

HosePL commented May 25, 2024

@courville I didn't notice you released new version two hours ago.
In 6.2.76 the issue I mentioned above is fixed, but now Nova again choose first embed subtitles instead of external (xxx-PL.srt) as default, also majority of videos which have polish embeded subtitles shows first subtitle and not polish.

@courville
Copy link
Contributor

courville commented May 26, 2024

@HosePL, regarding your remark about selecting the first embedded subtitles, do you mean it selects the polish embedded subtitle over the polish external one? This, I believe, I have not changed. However if it selects another language instead, this is a bug. Please confirm.

EDIT: no need to reply, I can reproduce

courville added a commit to nova-video-player/aos-Video that referenced this issue May 26, 2024
@courville
Copy link
Contributor

@HosePL
Copy link

HosePL commented May 26, 2024

@courville Just tested 6.2.77, after update I've cleared cache, program data of the app on TV and configured everything from scratch. I also deleted *.resume files on my SMB server.

Results:

  • embeded (polish) and no external - displayed properly (embeded polish)
  • embeded (no polish) and external (xxx-PL.srt) - external detected properly (on the list), but displayed first embed
  • embeded (polish) and external (xxx-PL.srt) - external detected properly (on the list), displayed properly (embeded polish)
  • no embeded and external subtitles (xxx-PL.srt) - external detected properly (on the list), displayed properly (external polish)

I also noticed that external language name on the list is "polski" and sometimes "Polski" (first capital letter).

@courville
Copy link
Contributor

courville commented May 26, 2024

Thanks for the feedback.

  • embeded (no polish) and external (xxx-PL.srt) - external detected properly (on the list), but displayed first embed

I just tested this very case tvshow-s01e01.mkv (3 internal subs en, fr, fr forced) with tvshow-s01e01-PL.mkv as external sub. With preferred displayed sub language Polish, on my side it displays correctly the subtitle.

What is the language (default locale) you use on your device?

@HosePL
Copy link

HosePL commented May 26, 2024

What is the language (default locale) you use on your device?

Polish

@HosePL
Copy link

HosePL commented May 27, 2024

@courville I've made some more tests this morning and here what I found:

Exact the same movie and subtitle files with different file names.
Embeded: english, japanese, romanian
External: polish

John.Wick.Chapter.4.2023.2160p.WEB-DL.DDP5.1.Atmos.DV.HDR10+.H.265-CMRG.mkv
John.Wick.Chapter.4.2023.2160p.WEB-DL.DDP5.1.Atmos.DV.HDR10+.H.265-CMRG-PL.srt

  • External polish language detected properly, displayed polish subtitles

John Wick 4.mkv
John Wick 4-PL.srt

  • External polish language detected properly, displayed first embeded subtitles

I believe you can easly reproduce this fault and fix it.

courville added a commit to nova-video-player/aos-Video that referenced this issue May 27, 2024
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

No branches or pull requests

4 participants