[gstreamer-bugs] [Bug 360246] [audioconvert] Optionally apply dithering

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=360246

  GStreamer | gst-plugins-base | Ver: HEAD CVS




------- Comment #15 from Sebastian Dröge  2007-04-16 18:46 UTC -------
This would require that we initialize GRandom always with the same value,
that's fine.

Still I would prefer to have everything in audioconvert and some parameters to
set it all up and then have sane defaults. Otherwise we would have nearly the
same element twice somewhere :/

Where did you get your coefficients for noise shaping from btw?


And you thought about the refactoring like this?
unpack: blow everything up to 32 bit ints (or doubles)
mix: mix the channels
quantize: add dithering and noise shaping, correctly round the values and set
the lower n bits to zero while still working on 32 bit ints
pack: shift and write to the target format


Using 64 bit integers as intermediate format sounds like a large performance
degradation though but would make everything much easier. I guess the best
would be to stay with 32 bit ints/doubles as intermediate format until
everybody uses a 64 bit architecture or what do you think? :)


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=360246.