[Milhouse]

I'll include the PR when it's available! Thanks h.udo.

[Milhouse]

New OpenELEC Helix build: #1117
(Supercedes previous build)

Code:

# uname -a
Linux rpi512 3.17.3 #1 PREEMPT Mon Nov 17 21:04:22 GMT 2014 armv6l GNU/Linux

# vcgencmd version
Nov 13 2014 17:21:14
Copyright (c) 2012 Broadcom
version 4b43a6c1fb32bd23b26f999eef78312d71df9b7a (clean) (release)

# lsb_release
OpenELEC (Milhouse) - Version: devel-20141117210305-r19653-g7beb180 [Build #1117]

Based on tip of OpenELEC master (7beb1804, changelog) and tip of XBMC master (17ea0454, changelog) with the following modifications:

• Includes newclock4 patches
  
• Excludes the OpenELEC fernetmenta patches due to conflicts with newclock4
  
• Excludes the OpenELEC linux-01-RPi_support patch in favour of sourcing these and possibly more recent patches directly from kernel branch rpi-3.17.y
  
• Excludes the OpenELEC xbmc-001-newclock4 patch in favour of sourcing these and possibly more recent patches directly from newclock4 branch
  
• Default setting for "Show RSS Feed" changed to disabled (new installs only) [patch details]
  
• Disabled "Total Duration" in Confluence (see build #0221 for details)
  
• Adapted service.openelec.settings to take advantage of PR:5217 [patch details]
  
• Includes latest libnfs master (4a58e614)
  
• Includes latest libcec master (70e1b5de)
  
• Includes latest xbmc-pvr-addons master (06842b90)
  
• Includes latest xbmc-addon-xvdr master (328fa653)
  
• Includes CONFIG_COREDUMP=y to allow creation of coredumps (see here)
  
• Includes additional ffmpeg codecs/muxers enabled for testing/benchmarking purposes (see patch)
  
• Includes commits from libcec-2.2.0 (popcornmix)
  
• Includes PR:5573: webserver: improved caching control (see discussion)
  
• Includes PR:5713: Fix "Current fanart" not being shown in DialogVideoInfo.
  
• Includes PR:5734: [videoinfoscanner] fix video_ts folder not stacked on movie scan
  

Build Highlights:

1. Samba/IFO/ISO/POSIX fixes merged
   
2. Possibility to clean databases in background
   

Build Details:

1. XBMC:
   • [dvdplayer] Remove superfluous cast. (PR:5736, 1 commit, 1 file changed)
     
   • Even better emulate POSIX file functions (reduced) (PR:5707, 14 commits, 3 files changed)
     
   • [pvr] fix selecting last playing group 'All TV/Radio channels' (PR:5733, 2 commits, 2 files changed)
     
   • [osx] - mark removable shares which are either have DVD or CD media in i... (PR:5738, 1 commit, 1 file changed)
     
   • [Fix] Reading zero bytes from a file should yield zero read bytes. (PR:5743, 1 commit, 1 file changed)
     
   • [pvr] fix marking recordings as watched or unwatched (PR:5613, 5 commits, 3 files changed)
     
   • [WIP] Fix for libdvdcss partial read (PR:5700, 3 commits, 3 files changed)
     
   • Samba fixes and cleanup for helix (PR:5740, 11 commits, 7 files changed)
     
   • [pvr] go to parent folder after deleting the last item in a subfolder (PR:5725, 2 commits, 4 files changed)
     
   • [emufnc] fix possible compiler warning (773a2d16)
     
   • [emufnc] return exact '-1' instead of other negative values in dll_read() and dll_write() (6634af39)
     
2. newclock4:
   • New commits in this build:
     • changed: Rename confusing CSimpleDoubleCache class to CDoubleCache (2d943992)
       
     • fixed: FileCache-full logic was wrong + implement CanWriteToCache() function (f424dd91)
       
     • changed: Use m_StateInput else dvdplayer osd info is 8 seconds delayed (6c1f070a)
       
     • fixed: Calling WaitForData with full cache caused it to always block the max. of 5 seconds (c38e1f50)
       
     • - fix premature close of pipe. It should only be closed if refcount is zero (b9cf7b7b)
       
     • video library: add the possibility to clean the database in the background (e00f3ba5)
       
     • music library: add the possibility to clean the database in the background (7b3c5326)
       
   • Commits no longer in build:
     • do database clean on thread (f81c5dea)
       
     • use extended progress bar for library clean (5d381499)
       
     • fixed: FileCache-full logic was wrong + create seperate IsCacheFull() functions (28b22302)
       
     • changed: Show cachebytes from input in dvdplayer info as it's more intuitive (72078b7f)
       

[h.udo]

Yes, thats it - Polish QWERTY (not QWERTZ!). Feel free to PM me.

Have a look here and let me know.

I probably missed something important (followed online Polish keyboard layout).

After OK I'll PR.

[allan87]

Every milhouse build following the Chapter skipping fix exhibits the same issue, whereas openelec beta 2 works correctly on all videos I have tested.

Does that help?

Not enough. I've tried a dozen m4v/mp4 films and all have either no chapters or chapters that seek correctly.
A sample file that is known to produce the issue is really required.

OK. I'll see what I can do without uploading something huge.

[Milhouse]

New OpenELEC Helix build: #1118
(Supercedes previous build)

Code:

# uname -a
Linux rpi512 3.17.3 #1 PREEMPT Tue Nov 18 21:36:24 GMT 2014 armv6l GNU/Linux

# vcgencmd version
Nov 13 2014 17:21:14
Copyright (c) 2012 Broadcom
version 4b43a6c1fb32bd23b26f999eef78312d71df9b7a (clean) (release)

# lsb_release
OpenELEC (Milhouse) - Version: devel-20141118213503-r19661-g548cc21 [Build #1118]

Based on tip of OpenELEC master (548cc218, changelog) and tip of XBMC master (9dae1b59, changelog) with the following modifications:

• Includes newclock4 patches
  
• Excludes the OpenELEC fernetmenta patches due to conflicts with newclock4
  
• Excludes the OpenELEC linux-01-RPi_support patch in favour of sourcing these and possibly more recent patches directly from kernel branch rpi-3.17.y
  
• Excludes the OpenELEC xbmc-001-newclock4 patch in favour of sourcing these and possibly more recent patches directly from newclock4 branch
  
• Default setting for "Show RSS Feed" changed to disabled (new installs only) [patch details]
  
• Disabled "Total Duration" in Confluence (see build #0221 for details)
  
• Adapted service.openelec.settings to take advantage of PR:5217 [patch details]
  
• Includes latest libnfs master (4a58e614)
  
• Includes latest libcec master (70e1b5de)
  
• Includes latest xbmc-pvr-addons master (06842b90)
  
• Includes latest xbmc-addon-xvdr master (328fa653)
  
• Includes CONFIG_COREDUMP=y to allow creation of coredumps (see here)
  
• Includes additional ffmpeg codecs/muxers enabled for testing/benchmarking purposes (see patch)
  
• Includes commits from libcec-2.2.0 (popcornmix)
  
• Includes PR:5573: webserver: improved caching control (see discussion)
  
• Includes PR:5713: Fix "Current fanart" not being shown in DialogVideoInfo.
  
• Includes PR:5734: [videoinfoscanner] fix video_ts folder not stacked on movie scan
  
• Includes PR:5747: [database] - add setting for hiding watched movies/episodes in recently added lists
  
• Includes PR:5752: [jsonrpc] Allow retrieving empty TV shows via VideoLibrary.GetTVShows.
  

Build Highlights:

1. [omxplayer] Fix for EAC3 passthrough
   
2. Added PR:5747 and PR:5752
   

Build Details:

1. OpenELEC:
   • kmod: update to kmod-19 (19ca282f)
     
   • scripts/install: install modprobe.d files to /lib (63508f3d)
     
   • kmod: symlink /etc/modprobe.d to /storage/.config/modprobe.d (f1b09e6a)
     
   • installer: remove beta warning (548cc218)
     
2. XBMC:
   • LinuxHWDecoders: Adjust to new component logging (PR:5695, 5 commits, 6 files changed)
     
   • [Fix][win32] Discs that were present at start are not removed from views... (PR:5750, 1 commit, 1 file changed)
     
   • library: don't remove the source if the user has changed his mind (PR:5753, 1 commit, 1 file changed)
     
   • [vfs/Pipe] - fix premature close of pipe. It should only be closed if (PR:5741, 1 commit, 1 file changed)
     
   • [pvr] Fix recording history (last directory not selected) (PR:5742, 2 commits, 3 files changed)
     
   • [python] add legacy deps. loading as fallback when running standalone script (PR:5748, 1 commit, 1 file changed)
     
   • video library: add the possibility to clean the database in the background (PR:5735, 2 commits, 8 files changed)
     
   • [woa] dont abort if wait for net fails (PR:5341, 1 commit, 1 file changed)
     
   • [addons] add toggle for hiding broken addons. hide by default (PR:5749, 1 commit, 2 files changed)
     
   • Revert "[pvr] Fix recording history (last directory not selected)" (PR:5754, 1 commit, 3 files changed)
     
   • [jenkins/darwin] - ensure that audioencoder addons are not removed from tree during cleanup (same like with pvr addons before) - fixed missing audioencoder addons in nightlies. (3ea82c9b)
     
   • [pydocs] cosmetic fix on line ending (baa4dd34)
     
3. newclock4:
   • New commits in this build:
     • [omxplayer] Fix for EAC3 passthrough (64a4b1f3)
       
   • Commits no longer in build:
     • VAAPI: Use Component Logging (717a2527)
       
     • VDPAU: Component Logging (f15c199e)
       
     • IMX: Use Component Logging (68c19bf5)
       
     • - fix premature close of pipe. It should only be closed if refcount is zero (b9cf7b7b)
       
     • video library: add the possibility to clean the database in the background (e00f3ba5)
       
     • music library: add the possibility to clean the database in the background (7b3c5326)
       
4. Additional commits/pull requests not yet merged upstream:
   • Added: PR:5747: [database] - add setting for hiding watched movies/episodes in recently added lists
     
   • Added: PR:5752: [jsonrpc] Allow retrieving empty TV shows via VideoLibrary.GetTVShows.
     

[popcornmix]

New OpenELEC Helix build: #1118

One feature I'd quite like testing is using dvdplayer (i.e. disable omxplayer acceleration), with "adjust display refresh rate to match video" and "sync playback to display".

On other platforms, the recommended A/V sync method is "resample audio".
This is required as audio and video devices may have asynchronous clocks, and will gradually drift apart causing a lipsync error. Correction may also be needed due to errors in the timestamps in the file (e.g. some rounding issue in the encoding tools) or after a glitch (e.g. error in broadcast TV, or after removing adverts from a file).

To correct this, you have to do something unwanted:
You can skip/duplicate a video frame. You may notice a stutter on the video.
You can skip/duplicate audio samples. You may notice an audio glitch when this occurs.
You can resample the audio by a small amount to speed up or slow down the audio to catch up with video. This is considered the best option, but it still degrades audio, and can't be used with passthrough (which falls back to duplicate/drop).

On the Pi there is a new option "Adjust PLL". This avoids all the unwanted behaviours, and corrects audio/video sync by subtly altering the HDMI pixel clock (though a PLL).
This should allow perfect A/V sync with no video or audio artifacts.

Try it. The codec info ('o' key) will show the value of the PLL (1.00000 would be normal, 1.00015 would be running fast to decrease the audio offset, and 0.99985 would be running slow to increase the audio offset). The "err:" value is how far off audio and video are. It should *slowly* move towards zero. I've made the PLL adjustment pretty slow for now, so it may take a minute or two to get to zero. Some TV/receivers will drop-out if the PLL is moved too quickly.

Currently there is a bug that means audio/video don't start in sync (affects all platforms), so you often see ~-50ms offset at start of video, but that should quickly reduce. FernetMenta is aware of this, so hopefully that will be a fix for this soon.

So, give it a try and report back if there are any issues with video smoothness, any audio glitches or out-of-sync issues, or any tv/receiver drop-outs. Also report if the "err:" term ever fails to end up close to zero.

[Milhouse]

Also report if the "err:" term ever fails to end up close to zero.

"error" seems to be fixed at 25% although audio doesn't appear to be out of sync (HDMI 2.1). The display is set to 1080p@60Hz. Mediainfo.

"Adjust display refresh rate to match video" is set to "Always", and "A/V sync method" is "Adjust PLL".

"Output configuration" is "Optimized", "Resample quality" is "GPU accelerated".

[eikman2k]

if i scan my folder with my TV Episodes, Kodi don´t found the new TV Shows and Episodes, but when i go manually to the folder and click on TV Show Information, Kodi will recognize the TV Show and add the TV Show to the libary..

What is wrong?

[popcornmix]

"error" seems to be fixed at 25% although audio doesn't appear to be out of sync (HDMI 2.1). The display is set to 1080p@60Hz. Mediainfo.

It's not error but err. You don't appear to have "adjust PLL" enabled.
Second line of codec info should include "pll: 1.00000, err: 1ms"

[Milhouse]

"error" seems to be fixed at 25% although audio doesn't appear to be out of sync (HDMI 2.1). The display is set to 1080p@60Hz. Mediainfo.

It's not error but err. You don't appear to have "adjust PLL" enabled.
Second line of codec info should include "pll: 1.00000, err: 1ms"

Hmmm... not getting that line in my OSD.

These are my current settings - they produce the Codec Info as per my previous post:

Video Playback:

Video Acceleration:

Video Output:

Audio Output:


I'll try a fresh guisettings.xml in case something there is messed up.

[Milhouse]

Wow, this is annoying. I reset guisettings.xml - I had the "pll" entry in Codec Info. I set a few more options, restarted Kodi and now it's gone again. Grrr...

[popcornmix]

Wow, this is annoying. I reset guisettings.xml - I had the "pll" entry in Codec Info. I set a few more options, restarted Kodi and now it's gone again. Grrr...

Did it disappear when changing a setting or just the reboot?
It's possible there is another setting that disables it (but I can't think of it).

[Milhouse]

OK...
1. systemctl stop kodi; rm guisettings.xml; systemctl start kodi
2. Enable Playback and Acceleration settings as per above.
3. Play video - pll is present in Codec Info
4. Stop video.
5. Play video - pll is no longer present in Codec Info

Something weird is going on...

This is my guisettings.xml that worked once: pastebin

[Milhouse]

I think it may have something to do with resuming a video - if I start from the beginning I have pll, if I resume I don't have pll.

Edit: Even that isn't consistently true. "pll" seems to be present fairly consistently on SD material, but tends to appear only once for an HD video then disappear on subsequent plays of the HD video. However if I then play an SD video and then re-play the HD video, the pll will be present again on the HD video.

Edit2: I just started/stopped an HD video 5 consecutive times - twice it had pll, 3 times it didn't have pll. Maybe some sort of timing/synchronisation issue that prevents pll from being enabled reliably?

[Milhouse]

I've been watching an episode and after several attempts at playback I have pll displayed.

The value for pll is 0.99985, however "err" is constantly increasing as follows (every 2 seconds):

-0.8ms then -2.5ms then -4.4ms, -6.4ms, -8.2ms, -10.0ms, -11.8ms ... -88.6ms, -90.3ms, -92.0ms, -93.7ms, -95.4ms, -97.1ms, -98.8ms

and then it starts again from -0.8ms. There does appear to be a visible audio sync issue with this video.