-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Enable AC-4 audio codec if supported #7329
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
base: master
Are you sure you want to change the base?
Conversation
Cloudflare Pages deployment
|
|
|
A .wgt Tizen build for testing (based on v10.11) is available from https://github.com/kinke/jellyfin-tizen-builds/actions/runs/19298759849. |
Tested a few ac4 files in mp4 container. It is direct playing but there is no audio. Also for 5.1.4 atmos ac4 track jellyfin media info shows as ac4 with 0 channels. Standard 5.1ch ac4 track is showing correctly but all tracks have no audio when played.Tested with Tizen 8 TV. |
|
The first Dolby sample I've linked plays fine on my Tizen 9, the thunder is audible. But I'm not going to test much more, since my personal interest is basically 0. - Note that the media info stuff comes from the server, that's unrelated to the web client here. |
I tested the first demo file too but don't know what causes the no audio issue. The file plays with audio if i play it via the native tizen media player via dlna. Since you are saying tizen 9 plays fine I couldn't figure out the exact issue with my tv. Btw thanks for your efforts and response. |
|
I've also tested https://ott.dolby.com/OnDelKits/AC-4/Dolby_AC-4_Online_Delivery_Kit_1.5/Test_Signals/muxed_streams/MP4/Example/Audio_ID_720p_50fps_h265_514ch_192kbps_ac4.mp4 now, with a 5.1.4 channel config. It's direct-played and audible just fine (FWIW, both on GQ65S95F alone and via AV receiver); media info shows 0 channels, playback info has no channels info either (but shows the same 192 kbps and 51200 Hz). Maybe it's really a Tizen 9 change. How about the .ts, are you seeing the same regression too, not playable at all anymore? |
|
After just discovering @dmitrylyzo's rather old So far, after testing a few samples, it surprisingly seems to be almost usable on my Tizen 9.
It'd be interesting to know whether AVPlay fixes the AC4 audio on your Tizen 8. |
That's really good🤩.. tested your avplay build and i can now directly play ac4 files with audio.. yeah subtitles doesn't work..for that anamorphic video..i saw some "is anamorphic conditions" in avplayvideoplayer.js file..maybe we could tweak that?..for .ts ac4 playback it says unsupported file even in the native tizen media player...this is some firmware or hardware level limitations?...i hope u will find something for that subtitles issue😊...also it would be nicer if we focus in AVplay API for video playback in tizen client.. |
|
That branch is not ready yet, because we need a subtitle overlay class for rendering ASS, PGS, and SRT (to support our styling settings). And client settings so that we can choose a player. |
|
@dmitrylyzo: Thx for the work and the response! Did I miss any PR, or is it really just that 'hidden' branch of yours? I've just found it incidentally by googling...
I've removed those is-not-anamorphic-conditions for AVPlay too (in that build), enabling direct-play; but the aspect ratio isn't applied automatically, unlike for the HTML5 player. So that's something that's worse in AVPlay, and after a quick look at the (crappy) API docs, I'm not sure the aspect ratio can be freely customized at all. I'm almost 100% happy with the HTML5 player on Tizen 9 (now after the latest 10.11 regression fixes); the only slight annoyance so far in practice is the need to remux for a secondary audio track (which apparently might have worked for Tizen < 8, so shame on Samsung if that's really true). So I suspect I wouldn't really gain a lot from switching to AVPlay; the situation might be different for older Tizen devices, where Samsung might have kept more features AVPlay-exclusive, like that AC-4 support apparently. |
|
Yeah i am too sticking with HTML5 player on my TV. AVplay API is good but can't trade off subtitles support for ac4 which will not be necessary as for now. So the current bugs in AVplay is 1.Subtitles Hoping for this to be solved if further development continues. @kinke So is this PR going to be closed or stay as a Draft? |
|
I've continued the AVPlay experiment somewhat, hacking in some subtitles support along the way: #7339
According to our 2 tests, we'd apparently have to restrict the HTML AC-4 support on Tizen to v9+, to prevent Tizen 8 from outputting no audio. Tests on older devices would be interesting. |
|
Hey, thanks for the build. Tested a few movies with subtitles. embedded subtitles works perfectly but external subtitles doesn't seem to work. Also, I noticed a bug where the previous subtitle text stays in the screen until the next dialogue comes. It is not noticeable in continuous convos but highly noticeable during high gap dialogues and songs. Need someone to test this in older Tizen versions like 5,6,7. Update: External subtitles too works perfectly. The movie I tested had a corrupted subtitle. So the bug i mentioned is the only issue in subtitles rendering. |



Enable direct-play of AC-4 audio streams if the player claims support for it.
I've tested this on a Tizen 9 TV: