After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 301759 - [audioresample] float audio support (for OSX audio sinks)
[audioresample] float audio support (for OSX audio sinks)
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Mac OS
: Normal normal
: 0.10.7
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-04-24 01:15 UTC by Jonathan Matthew
Modified: 2006-04-28 14:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (1.64 KB, patch)
2005-04-24 01:20 UTC, Jonathan Matthew
none Details | Review
patch to reenable more audio formats (4.20 KB, patch)
2006-04-14 13:35 UTC, Wim Taymans
committed Details | Review

Description Jonathan Matthew 2005-04-24 01:15:28 UTC
Distribution/Version: 10.3.7

playbin currently can't play audio on OS X, because audioscale only handles
audio/x-raw-int and osxaudiosink only accepts audio/x-raw-float, so the link
fails and osxaudiosink just sits there being sad.

Enabling passthrough of float data makes it work in at least the common cases. 
It'd be nice if actual resampling worked too, but according to comments in the
code, there are some problems with that.  I haven't looked into it, in any case.
Comment 1 Jonathan Matthew 2005-04-24 01:20:08 UTC
Created attachment 45602 [details] [review]
proposed patch

Tested on OS X and linux, works for me
Comment 2 Ronald Bultje 2005-04-25 14:19:58 UTC
Applied. The other audioscale bug is already somewhere else, let's keep that one
(#160894) open.
Comment 3 Tim-Philipp Müller 2005-05-19 14:44:24 UTC
I think this patch should be reverted, as it appears to cause regressions for
programs still using spider (or spider fixed of course, if it is a spider bug).

I now get something like this when trying to extract tags from an ogg/vorbis file:

sceptic [tim] - ---> scentric-import --tags-only foo.ogg --debug
:: debug :: -----------------------------------------------------------------
:: debug :: Processing 'foo.ogg'. size = 144090169, itemid = 0
:: debug :: Metadata pipeline: filesrc ! typefind ! spider ! audio/x-raw-int !
fakesink
:: debug :: Metadata pipeline playing...
:: debug :: Metadata pipeline element added by spider: oggdemux0
:: debug :: Metadata pipeline mime type: application/ogg (NEW)
:: debug :: Metadata pipeline element 'oggdemux0' found tag.
:: debug :: Metadata pipeline element added by spider: vorbisdec0
:: debug :: Metadata pipeline mime type: audio/x-vorbis (NEW)
:: debug :: Metadata pipeline element added by spider: audioscale0
:: debug :: Metadata pipeline element 'vorbisdec0' found tag.

** (scentric-import:22899): CRITICAL **: gst_audioscale_link: assertion `caps'
failed
ERROR (0x805b0e0 - 310142:25:25.968476000)        GST_PADS(22899)
gstpad.c(2562):gst_pad_set_explicit_caps:<vorbisdec0> failed to negotiate
(try_set_caps with "audio/x-raw-float, rate=(int)44100, channels=(int)2,
endianness=(int)1234, width=(int)32, buffer-frames=(int)0" returned REFUSED)
:: debug :: Metadata pipeline error: Internal GStreamer error: pad problem. 
File a bug.
ERROR (0x805b0e0 - 310142:25:25.969236000)       scheduler(22899)
gstoptimalscheduler.c(2798):gst_opt_scheduler_iterate:<optscheduler0> in error state
:: debug :: Metadata pipeline done. No EOS. No hand-off. Got error.


http://mail.gnome.org/archives/rhythmbox-devel/2005-May/msg00072.html

looks like the same issue.

Cheers
 -Tim
Comment 4 Thomas Vander Stichele 2005-05-20 13:16:17 UTC
removing this patch for the release, this needs more work
Comment 5 Andy Wingo 2005-07-16 12:48:06 UTC
should probably just get audioscale to do float, or use audioresample
Comment 6 Andy Wingo 2006-01-27 17:28:43 UTC
The audioresample element still only does int. Tough to know what the real issue is until osxaudiosink gets ported.
Comment 7 Wim Taymans 2006-04-14 13:35:16 UTC
Created attachment 63461 [details] [review]
patch to reenable more audio formats

adding the other supported formats to the caps and activating them seems to work fine for me.
Comment 8 Wim Taymans 2006-04-28 14:17:11 UTC
        * gst/audioresample/gstaudioresample.c: (gst_audioresample_init),
        (resample_set_state_from_caps):
        Add support for other formats audioresample can handle such as
        32 bits in and float and 64 bits float. Fixes #301759