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

When starting an episode of series there are sound problems with the Disney+ addon #1335

Closed
Ronny-nerd opened this issue Jul 21, 2023 · 30 comments · Fixed by #1364
Closed

When starting an episode of series there are sound problems with the Disney+ addon #1335

Ronny-nerd opened this issue Jul 21, 2023 · 30 comments · Fixed by #1364

Comments

@Ronny-nerd
Copy link

Bug report

Describe the bug

Here is a clear and concise description of what the problem is:

Hardware: Nvidia Shield TV 2015 (Android TV11)
No AVR and no soundbar
InputStream Adative version 21.1.3

Kodi 21 Omega Alpha 2
With InputStream Adaptive version 21.1.0 and Kodi 20.2 Nexus with InputStream Adaptive version 20.3.11 there is not this problem.

When starting an episode of series there are sound problems with the Disney+ addon and for example with these series.

Secret Invasion
The Mandalorian
Obi-Wan Kenobi
The Book of Boba Fett
Star Wars: Andor
Star Wars: Tales of the Jedi

For me, there is no sound at the start of an episode in said series, only when you manually switch the sound.

Translated with www.DeepL.com/Translator (free version)

Manifest-Problem 3 InputStream Adaptive.zip

072123115009

072123115308

Sorry my english is no good.

@CastagnaIT
Copy link
Collaborator

you need to disable the dis€y+ manifest alteration proxy
and then provide new logs and manifests

@Ronny-nerd
Copy link
Author

Ronny-nerd commented Jul 21, 2023

Sie müssen den Manifeständerungs-Proxy von dis€y+ deaktivieren

And where can you set the Disney+ Manifest Proxy to disable in the Disney+ Addon?

@CastagnaIT
Copy link
Collaborator

CastagnaIT commented Jul 21, 2023

i think you have to open "SlyGuy common addon" and disable "slyProxy enabled" on advanced settings

@xbmc xbmc deleted a comment from Ronny-nerd Jul 21, 2023
@Ronny-nerd
Copy link
Author

Ronny-nerd commented Jul 21, 2023

Thanks for the info...CastagnaIT.

@Ronny-nerd
Copy link
Author

Manifest-Problem 4 InputStream Adaptive.zip

No sound after starting an episode.

072123183617
072123183634
072123183649

Sound only came after switching manually.

072123184257
072123184310

@Ronny-nerd
Copy link
Author

Ronny-nerd commented Jul 22, 2023

What surprises and irritates me is that a movie with InputStream Adaptive version 21.1.3 starts and runs without sound problems.

@CastagnaIT
Copy link
Collaborator

CastagnaIT commented Jul 22, 2023

can you stop add screenshots? have no utility and are making this thread a mess
the only thing that could have sense is provide the manifests of a working video in order to try find differences

from what i can see seem there is a wrong url where miss /r/ on address
thats weird because the code its the same for all

@CastagnaIT
Copy link
Collaborator

i rechecked whole code around download urls and also tried put manifests on a new test build
but i havent found nothing that can explain this wrong url

@matthuisman are you able to replicate this problem?

@glennguy
Copy link
Contributor

@CastagnaIT I see what you're seeing - the variant playlist is in a subfolder of the master playlist, but the segments seem to be calculated against the path of the master instead of the variant (missing /r folder).

@CastagnaIT
Copy link
Collaborator

exact, also we can see that baseurl on representation is set always from the effective url
https://github.com/xbmc/inputstream.adaptive/blob/Omega/src/parser/HLSTree.cpp#L212
and on download since that segment file have a relative the path, path is joined with the same representation baseurl:
https://github.com/xbmc/inputstream.adaptive/blob/Omega/src/common/AdaptiveStream.cpp#L269-L270
i was thinking of a bug of URL::Join method but testing it given right output

@MauriceW67
Copy link

@matthuisman are you able to replicate this problem?

I am not @matthuisman but I can confirm this issue also happens with my setup. I think it has something to do with enabling or disabling Atmos sound in the addon. If I switch audio tracks while playing, I can choose between DD+ and DD+ Atmos, and the first one does produce sound.

Another issue I have (which doesn't happen with Kodi 20.2 and IS:A 20.3.11) is that the subtitles that are selected (Dutch in my case) do not appear.

@CastagnaIT
Copy link
Collaborator

CastagnaIT commented Aug 12, 2023

try test last nightly
https://jenkins.kodi.tv/blue/organizations/jenkins/xbmc%2Finputstream.adaptive/detail/Omega/93/artifacts
let's see if it's a side effect of memory corruption now fixed

@MauriceW67
Copy link

@CastagnaIT Unfortunately that test version did not make any difference.

@Ronny-nerd
Copy link
Author

For me the same result with this build as for MauriceW67.

Problem InputStream Adaptive 21.1.3 - Kodi Omega - Disney Addon.zip

@GG-76
Copy link

GG-76 commented Aug 20, 2023

Hi, I have the same version of both Kodi (Omega) and InputStream (21.1.3). When playing some series on Disney+, I have the same problem, a "broken" picture, non-functional Dolby Atmos sound, subtitles not appearing in my language, etc. At the same time, but when I play, for example, the movie Guardians of the Galaxy 3, which has the same parameters, there is no problem at all. Also, the add-on Osmosis for exporting streams to the Kodi library stopped working correctly for me with Disney+ (not completely, but for example when I wanted to add the new High School Musical series), which may not be related, but there was no problem with this add-on before, and it still works with other add-ons without problems.

@CastagnaIT
Copy link
Collaborator

CastagnaIT commented Aug 25, 2023

@Ronny-nerd @MauriceW67 can you give me a feedback with following test build?

PS. is required last Kodi nightly due to API change

@CastagnaIT
Copy link
Collaborator

@MauriceW67
Copy link

@CastagnaIT I justed tested this build with the latest kodi nightly and the Disney+ addon.

My findings sofar:

  • Video plays fine in 4K / Dolby Vision.
  • Audio plays, but I have to manually select the correct stream. If I enable Atmos in the addon, it does choose the Atmos track (showing as 16 channels), but my receiver stays silent. If I disable Atmos in the addon, it defaults to an English AAC 2-channel track which also does not give any sound. Manually selecting the DD+ track does produce sound correctly.
  • Subtitles are a strange thing. Whatevery I do, subtitles in my own language (Dutch) will not show, even though they show as being enabled in Kodi. Switching to another language works, but switching back to Dutch makes them disappear again.

Btw, it doesn't mattter if I setup default and allowed audio and subtitle languages within the addon or not. The odd behavior with the sound and subtitle tracks stays the same. Maybe @matthuisman can shed some light on this?

Please note that with Kodi 20.2 and IS:A 20.3.11 the issues with the audio and subtitles streams are not there.

If you need any logs, let me know.

@CastagnaIT
Copy link
Collaborator

CastagnaIT commented Aug 25, 2023

the most important thing is you need to make sure that manifest alteration proxy setting is set disabled from slyshguy addon

the default audio/subtitle track choosed, not depends from ISAdaptive
but without log what could i do, nothing... since any maintainer here dont have D+
it's already a mess to be able to figure it out without being able to replicate locally

the only thing i can theorize is that its possible that the log there are same errors due to same wrong url path without /r for unknown reason
in this case maybe having a log with enabled Curl logging from kodi logging component settings maybe could help show something more

@matthuisman
Copy link
Contributor

@MauriceW67
Just focus on the single sound issue this ticket is for.

Confirm that is fixed with the test builds and still broken with current stable builds.

You can turn off slyguy proxy in the slyguy common settings (under services)

@Ronny-nerd
Copy link
Author

@matthuisman ...What about your test on this subject?

Have the same problems with your new test build from today...CastagnaIT.

Problem.zip

@CastagnaIT
Copy link
Collaborator

the log seem bit better but

2023-08-26 00:10:38.569 T:19685   error <general>: AddOnLog: inputstream.adaptive: Open: CURLOpen failed
2023-08-26 00:10:38.569 T:19685   error <general>: AddOnLog: inputstream.adaptive: [AS-259] Download failed, internal error: b07ff016-7c04-4915-9990-581f5620c3c5/524c-MAIN/03/audio_ec3_256K/map.mp4a

however again a bad path
i found something e.g.

#EXT-X-MEDIA:TYPE=AUDIO,GROUP-ID="atmos",NAME="Deutsch",LANGUAGE="de",AUTOSELECT=YES,CHANNELS="6",URI="r/composite_256k_ec-3_de_PRIMARY_a39c570a-3c46-4db6-b42f-751c28f17566_10310d0e-3231-46ca-8ec4-822aec4d1f6b.m3u8"

#EXT-X-MEDIA:TYPE=AUDIO,GROUP-ID="eac-3",NAME="Deutsch",LANGUAGE="de",AUTOSELECT=YES,CHANNELS="6",URI="r/composite_256k_ec-3_de_PRIMARY_a39c570a-3c46-4db6-b42f-751c28f17566_10310d0e-3231-46ca-8ec4-822aec4d1f6b.m3u8"

two renditions that point to the same manifest, now i added code to ignore duplicates that i havent taken in account on rework
the manifest call atmos a track that isnt atmos...good mixed things by luck at least the "joc" param is missing so not a problem

i noticed also that there same audio tracks with multiple bitrate like:

#EXT-X-MEDIA:TYPE=AUDIO,GROUP-ID="aac-128k",NAME="Español",LANGUAGE="es-ES",AUTOSELECT=YES,CHANNELS="2",URI="r/composite_128k_mp4a.40.2_es-ES_PRIMARY_017d7b89-cde0-49c9-964e-83887fdc30e0_4d3762ef-ee9a-4835-b76f-1eb9ec83ecb1.m3u8"

#EXT-X-MEDIA:TYPE=AUDIO,GROUP-ID="aac-64k",NAME="Español",LANGUAGE="es-ES",AUTOSELECT=YES,CHANNELS="2",URI="r/composite_64k_mp4a.40.2_es-ES_PRIMARY_d3fd986b-cdf6-4e01-bae2-1202a4e5539f_4d3762ef-ee9a-4835-b76f-1eb9ec83ecb1.m3u8"

this will be a bit problem for quality auto selection since not exist a BANDWIDTH or BITRATE HLS param
i think will be choosen the first in order of apparence in the manifest
however this not concern the current issue

Jenkins CI builds are currently dead on the repository
i will prepare a test build manually soon

@CastagnaIT
Copy link
Collaborator

here new test
inputstream.adaptive-android-aarch64-omega-decc930_1.zip
please disable Curl on logging its not more needed, also manifests are not more needed, just log file

@Ronny-nerd
Copy link
Author

Ronny-nerd commented Aug 26, 2023

Here is the Kodi Debug LOG file with your latest test build inputstream.adaptive-android-aarch64-omega-decc930_1.zip

Problem2.zip

The problem persists, even with InputStream Adaptive version 21.2.0

@MauriceW67
Copy link

MauriceW67 commented Aug 26, 2023

I also just tested with the decc930_1 version.

SlyProxy is disabled and Atmos is enabled in both Slyguy Common and the Disney+ addon.

When starting an episode, the addon selects the "DD+ ATMOS 16-channel" audio track, but no sound is played (note: the 3-second Disney+ bumper before the episode does play sound).

Manually switching to the normal DD+ track with 6 channels makes the sound play.

Debug log here:

kodi.log

With Atmos disabled, the behavior is slightly different. The addon then selects the AAC-LC 2-channel audio track as default, but no sound can be heard. Switching manually to the DD+ 6-channel track makes the sound play again.

Debug log here:

kodi.log

EDIT: Just noticed a second annoying issue. If the episode is played for a second time with Atmos disabled, the previous manually selected DD+ 6-channel track is apparently remembered, but no sound can be heard. Switching to the AAC-LC track that previously didn't play sound, now does play sound. But switching back to DD+ 6-channel makes everything go quiet again. So it seems that the audio track that is chosen first by the addon, never produces any sound, no matter if you switch back and forth to it.

@CastagnaIT
Copy link
Collaborator

ok i found how to rougly reproduce it
i think finally found the problem was on periods
the current code creates periods from EXT-X-DISCONTINUITY when parsing for the first time any child manifest
but the code set the baseurl to current representation only, then when switch to another period miss to set baseurl
the current code work flow is a bit messy also to understand...

test build, should works:
inputstream.adaptive-android-aarch64-omega-a16815f_1.zip

@Ronny-nerd
Copy link
Author

inputstream.adaptive-android-aarch64-omega-a16815f_1.zip = Top...thanks CastagnaIT ...the new build now works.

@MauriceW67
Copy link

New build works for me too. And subtitles also, so the other issue I created can also be resolved.

Thanks for all the work you are putting into this @CastagnaIT

@GG-76
Copy link

GG-76 commented Sep 4, 2023

@CastagnaIT Hi, I would like to ask if this is fixed in version 21.2.0. because I can't install the fix file here (it's for aarch64). Unfortunately, on the latest Nightly Kodi Omega with version 21.2.0, the video and subtitles work for me when playing from Disney+, but I still don't get the Atmos sound :( It only plays the first few seconds at the Disney+ logo and then nothing... Thank you

@matthuisman
Copy link
Contributor

the issue was fixed with PR: #1364
It was merged 5 days ago.

There is no new release yet that includes it.

But, you can just download the build of latest master from below:
https://jenkins.kodi.tv/blue/organizations/jenkins/xbmc%2Finputstream.adaptive/detail/Omega/100/artifacts

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 a pull request may close this issue.

6 participants