Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#12906 closed Bug Report - General (fixed)

Some mp3s no longer playable in mythmusic

Reported by: stuartm Owned by: paulh
Priority: major Milestone: 0.28.1
Component: Plugin - MythMusic Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

A very tiny percentage of mp3s in my collection no longer play in mythmusic, but play fine with mythffplay/mythavtest even when streamed from the storage group in the case of mythavtest.

2016-10-19 19:24:10.426339 E  RemoteAVFormatContext::Open: Failed to probe file: myth://Music@scafell/Alternative/Elbow/The Seldom Seen Kid/Elbow - The Loneliness Of A Tower Crane Driver.mp3
2016-10-19 19:24:10.426457 E  Decoder Error: Could not open url  (myth://Music@scafell/Alternative/Elbow/The Seldom Seen Kid/Elbow - The Loneliness Of A Tower Crane Driver.mp3)
2016-10-19 19:24:10.426832 N  Suspending idle timer
2016-10-19 19:24:10.445044 E  RemoteAVFormatContext::Open: Failed to probe file: myth://Music@scafell/Alternative/Calexico/Garden Ruin/Calexico - Panic Open String.mp3
2016-10-19 19:24:10.445184 E  Decoder Error: Could not open url  (myth://Music@scafell/Alternative/Calexico/Garden Ruin/Calexico - Panic Open String.mp3)

Change History (5)

comment:1 Changed 7 years ago by paulh

Try increasing the size of the probe buffer here:- https://github.com/MythTV/mythtv/blob/master/mythplugins/mythmusic/mythmusic/remoteavformatcontext.h#L56

The default is 64K which is should be plenty to probe all audio only formats but unfortunately we are seeing more files with the cover art in an MJPEG video stream which could mean the probe size will need to be adjusted to properly probe these files.

What does mythffprobe say about what streams are present?

comment:2 Changed 7 years ago by stuartm

I doubled the probe buffer (for the sake of getting a quick answer), it resolves the issue for all the affected tracks I knew about.

Here's what mythffprobe has to say for a couple of them:

  Duration: 00:05:14.49, start: 0.025056, bitrate: 215 kb/s
    Stream #0:0: Audio: mp3, 44100 Hz, stereo, s16p, 213 kb/s
    Metadata:
      encoder         : LAME3.97 
    Stream #0:1: Video: mjpeg, yuvj420p(pc, bt470bg/unknown/unknown), 600x519 [SAR 72:72 DAR 200:173], 90k tbr, 90k tbn, 90k tbc
    Metadata:
      title           : 600x600.jpg
      comment         : Cover (front)
  Duration: 00:04:09.78, start: 0.025056, bitrate: 190 kb/s
    Stream #0:0: Audio: mp3, 44100 Hz, stereo, s16p, 188 kb/s
    Metadata:
      encoder         : LAME3.96r
    Side data:
      replaygain: track gain - -6.800000, track peak - unknown, album gain - unknown, album peak - unknown, 
    Stream #0:1: Video: mjpeg, yuvj420p(pc, bt470bg/unknown/unknown), 600x600 [SAR 72:72 DAR 1:1], 90k tbr, 90k tbn, 90k tbc
    Metadata:
      comment         : Cover (front)

comment:3 Changed 7 years ago by Paul Harrison <pharrison@…>

Resolution: fixed
Status: newclosed

In e8d36304610f79ab6ec6042d0a251db83586f08f/mythtv:

MythMusic: increase the probe buffer size to 128K

There is a increasing trend to put the cover art images into an MJPEG video
stream which was sometimes causing the probing of some files to fail because
the buffer didn't contain enough data to determine the available streams.

Fixes #12906.

comment:4 Changed 7 years ago by Paul Harrison <pharrison@…>

In 3930f5d13a7e0a4b6328a1b3032374752dfacbe4/mythtv:

MythMusic: increase the probe buffer size to 128K

There is a increasing trend to put the cover art images into an MJPEG video
stream which was sometimes causing the probing of some files to fail because
the buffer didn't contain enough data to determine the available streams.

Fixes #12906.

(cherry picked from commit e8d36304610f79ab6ec6042d0a251db83586f08f)

comment:5 Changed 7 years ago by paulh

Milestone: unknown0.28.1
Note: See TracTickets for help on using tickets.