Bug 10036 – Metadata delay calculation does not take into account transcoded bitrates

Bug 10036 - Metadata delay calculation does not take into account transcoded bitrates

Summary:

Metadata delay calculation does not take into account transcoded bitrates

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Transcoding
Version:	7.3.0
Platform:	PC Other

Importance:	P3 normal (vote)
Target Milestone:	7.x
Assigned To:	Andy Grundman

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-11-17 14:48 UTC by Andy Grundman
Modified:	2009-07-31 10:32 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Andy Grundman 2008-11-17 14:48:44 UTC

For example, MP3 -> WAV transcoding, the delay calculation used for metadata returns very large values because it is using the MP3 bitrate instead of the transcoded bitrate.

http://forums.slimdevices.com/showthread.php?t=55167

Comment 1 Andy Grundman 2008-11-18 11:44:15 UTC

Alan: Since MP3 -> WAV is not one of the default configs, is this supported by $client->streamingSong()->streambitrate()?  Will that give the right value for bitrate?  This works at least for bitrate limiting so I'm going check that in for now.

Comment 2 Andy Grundman 2008-11-18 11:45:36 UTC

Change 23958.

Comment 3 Alan Young 2008-11-19 01:58:31 UTC

Andy, yes this should work fine. See S::P::Song::guessBitrateFromFormat().

Comment 4 Andy Grundman 2008-11-19 06:01:42 UTC

Great, marking fixed.

Comment 5 James Richardson 2008-12-15 12:09:44 UTC

This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.

Comment 6 Chris Owens 2009-07-31 10:32:11 UTC

Reduce number of active targets for SC