Bug 16341 - Need support for FLAC to AIFF transcoding
: Need support for FLAC to AIFF transcoding
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Transcoding
: 7.5.x
: PC Windows 7
: P2 normal with 4 votes (vote)
: 7.7.0
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-07-03 09:21 UTC by michael123
Modified: 2011-10-17 05:59 UTC (History)
4 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description michael123 2010-07-03 09:21:27 UTC
I've created following custom-convert.conf
----
# FLAC output starts here 

flc aif * *
	# FT:{START=--skip=%t}U:{END=--until=%v}D:{RESAMPLE=-r %d}
	[flac] -dcs $START$ $END$ -- $FILE$ | [sox] -v0.95 -q -t wav - -t aiff -b24 - rate -v 96k

flc aif transcode *
	# FT:{START=--skip=%t}U:{END=--until=%v}D:{RESAMPLE=-r %d}
	[flac] -dcs $START$ $END$ -- $FILE$ | [sox] -v0.95 -q -t wav - -t aiff -b24 - rate -v 96k
---

However, when I create the .AIFF file offline, I can play it just fine
Comment 1 michael123 2010-07-03 09:30:19 UTC
I see also following error in server.log

I am not if it is connected, but anyway..

[10-07-03 19:18:17.3485] Slim::Utils::Misc::msg (1165) Warning: [19:18:17.3480] Use of uninitialized value in numeric eq (==) at /<C:\PROGRA~1\SQUEEZ~1\server\SQUEEZ~3.EXE>Slim/Player/Squeezebox.pm line 652.


The build is v7.5.1, r30836
Comment 2 Brian S 2011-04-18 19:54:34 UTC
This also occurs with OGG Vorbis files, at least Squeezebox Server 7.5.1 through 7.6.0-32302 nightly.

Here is a related thread.
http://forums.slimdevices.com/showthread.php?t=83531

This occurs for me in both 64-bit Ubuntu 10.10 and Windows 7 x64 with a Roku Soundbridge (yeah, I know).

I believe this affects Squeezebox users just as much, though.

Regardless, streaming PCM doesn't seem to work very well - the wifi & buffer can't keep up with the high bandwidth. This used to work fine, say, a year ago.
Comment 3 Brian S 2011-04-18 20:08:55 UTC
Also, this may be a duplicate of bug 16532
Comment 4 Harald Alvestrand 2011-08-12 08:36:07 UTC
Happens in 7.6.0 with Linux (32 bit Intel) server too.
Workaround of disabling conversion to AIFF worked in 7.5, but results in file not being playable in 7.6.0.
Comment 5 Andy Grundman 2011-09-14 08:04:11 UTC
I've added a flc->aif convert.conf line in 7.7. Now that I think of it, this probably needs resampling though...


flc aif * *
        # FT:{START=--skip=%t}U:{END=--until=%v}
        [flac] -dcs --force-raw-format --endian=big --sign=signed $START$ $END$ -- $FILE$
Comment 6 Jim McAtee 2011-10-05 21:06:47 UTC
(In reply to comment #5)
> I've added a flc->aif convert.conf line in 7.7. Now that I think of it, this
> probably needs resampling though...
> 
> flc aif * *
>         # FT:{START=--skip=%t}U:{END=--until=%v}
>         [flac] -dcs --force-raw-format --endian=big --sign=signed $START$ $END$
> -- $FILE$

This doesn't work. I get only noise. Tested on a Touch and on a Boom.

What's the need for FLAC->AIFF streaming? Why wouldn't you just use PCM?
Comment 7 Jim McAtee 2011-10-05 21:22:30 UTC
The above rule also takes precedence over FLAC->PCM streaming, so that if someone has disabled native FLAC streaming with the intention of streaming FLAC as PCM, they'll get noise. And if it is fixed, you'd have many people who are no longer streaming FLAC as PCM until they've disabled the new rule.

This needs to be removed.
Comment 8 Harald Alvestrand 2011-10-05 23:44:34 UTC
Note - things started working for me once I installed LAME.
Apparently the slimserver tries to use the LAME command without checking that it's present first.
Comment 9 Mikael Nyberg 2011-10-15 08:48:25 UTC
(In reply to comment #7)
> The above rule also takes precedence over FLAC->PCM streaming, so that if
> someone has disabled native FLAC streaming with the intention of streaming FLAC
> as PCM, they'll get noise. And if it is fixed, you'd have many people who are
> no longer streaming FLAC as PCM until they've disabled the new rule.
> 
> This needs to be removed.

I concur FLAC>PCM ahould overule FLAC>AIFF
Comment 10 SVN Bot 2011-10-17 05:59:12 UTC
 == Auto-comment from SVN commit #33611 to the slim repo by agrundman ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33611 ==

Fixed bug 16341, don't set wrong endian flag if streaming transcoded to aif