Crash when recording and streaming OGG with a samplerate of 96000

[RAFFI TEA]

While MP3 streaming and recording is possible with a samplerate of 96000 this is not true for OGG, The encoder does not support downsampling to 44100 or 48000 as opposed to LAME.

Two option:
 1. We throw a message box and disable recording and shoutcast at 96000
 2. Downsampling? I'm sure we've some code for that somewhere in Mixxx

[Sean M. Pappalardo (pegasus-renegadetech)]

Can we just not show 96000 as an option if Vorbis is selected?

[Sean M. Pappalardo (pegasus-renegadetech)]

(BTW be sure to call it OGG Vorbis instead of just OGG, since an OGG file can technically contain a number of different codecs.)

[William Good (bkgood)]

Preferably not, having prefs in one pane (Sound Hardware) affected by prefs in
another pane (Streaming) is a total mess because of how the prefs are one-
class-per-pane (search for scratchlib in dlgprefsound.cpp). Even if they
weren't modular, it also adds confusion for the user: a user sees that 96kHz
is disabled and doesn't know why. You can tell them with an extra GUI label,
but that just adds clutter.

My personal preference would be to downsample the 96k samples to 48k with
libsamplerate in the engine sidechain thread.

On Monday, December 06, 2010 18:12:01 you wrote:
> Can we just not show 96000 as an option if Vorbis is selected?

[Sean M. Pappalardo (pegasus-renegadetech)]

Oh, I thought he was talking about stream bit rate. If he does indeed mean sample rate, then I too vote for downsampling so at least your main output still runs at 96kHz, incase you're playing at a venue AND streaming.

[RAFFI TEA (raffitea)]

I think downsampling is the best, too. Unfortunately, I've never worked with libsamplerate and my knowledge about audio processing is limited here. If someone other than me could implement the downsampling in EncoderVorbis::process(...) I would be thankful.

[RAFFI TEA (raffitea)]

Oh and just to be precise here: LAME does already downsample for us. So we only have to touch the EncoderVorbis class.

[RJ Skerry-Ryan (rryan)]

Pushing this to 1.10.1 since it's unlikely someone will hit this, and if they do they'll know immediately.

[RJ Skerry-Ryan (rryan)]

As a stopgap, I added an error dialog when someone tries to set 96kbps + vorbis to 1.9.x.

[RJ Skerry-Ryan (rryan)]

Also, I found out why this crashes. The immediate segfault is because we call ogg_stream_clear on a stream that has not been initialized. I added a boolean to indicate when we have initialized the ogg/vorbis data structures and that fixed the crash. It still can't stream, but at least it doesn't crash.

[RJ Skerry-Ryan (rryan)]

Marking fixed since this bug is about the crash. I'll file a wishlist bug for 96kHz ogg vorbis support.

[jus (jus)]

Also see Wishlist bug for 96kHz ogg vorbis support @ lp:883381

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/5701