playaudio
is broken!
It's worth reading the default implementation of playsound
(version 3.6.2):
function playaudio (name, ext)
if (nargin < 1 || nargin > 2)
print_usage ();
endif
if (nargin == 1 && isnumeric (name))
## play a vector
if (! isvector (name))
error ("playaudio: X must be a vector");
endif
X = name(:) + 127;
unwind_protect
file = tmpnam ();
fid = fopen (file, "wb");
fwrite (fid, X, "uchar");
fclose (fid);
[status, out] = system (sprintf ('cat "%s" > /dev/dsp', file));
if (status != 0)
system (sprintf ("paplay --raw \"%s\"", file))
endif
unwind_protect_cleanup
unlink (file);
end_unwind_protect
elseif (nargin >= 1 && ischar (name))
## play a file
if (nargin == 1)
name = [name ".lin"];
elseif (nargin == 2)
name = [name "." ext];
endif
if (any (strcmp (ext, {"lin", "raw"})))
[status, out] = system (sprintf ('cat "%s" > /dev/dsp', name));
if (status != 0)
system (sprintf ('paplay --raw "%s"', name))
endif
elseif (any (strcmp (ext, {"mu", "au" "snd", "ul"})))
[status, out] = system (sprintf ('cat "%s" > /dev/audio', name));
if (status != 0)
system (sprintf ('paplay "%s"', name))
endif
else
error ("playaudio: unsupported extension '%s'", ext);
endif
else
print_usage ();
endif
endfunction
There are some things to note:
- Writing directly to /dev/dsp always fails in the latest linux distros, so every time
you run the command you will get an error (on the
cat > /dev/dsp
line).
- It's hard-coded to use
paplay
, a command-line pulseaudio player.
- The
paplay
call will never work because paplay
defaults to s16ne
(probably a typo; I think they meant s16be
- signed 16-bit big endian), and playaudio
writes unsigned 8-bit!
- It calls it using
system()
. Always a bad idea.
- It writes the audio out to a file rather than streaming it. Might cause problems for large files.
- Unlike matlab, it doesn't handle floating point audio. It actually only supports 8-bit audio! Kind of stupid since that is the result returned by wavread!
- Unlike matlab, it only supports one sample rate (44100 Hz).
This function is extremely hacky, insecure and unreliable. If it in any way represents the code quality elsewhere in Octave... well, that is worrying. It should really really be reimplemented as proper function in Octave using portaudio.
Slightly better version
I don't really have time or motivation to do a lot of hacking on octave, so in the mean time I suggest you instead use this slightly better function:
function playsound(wav, samplerate)
# Play a single-channel wave at a certain sample rate (defaults to 44100 Hz).
# Input can be integer, in which case it is assumed to be signed 16-bit, or
# float, in which case it is in the range -1:1.
if (nargin < 1 || nargin > 2)
print_usage();
endif
if (nargin < 2)
samplerate = 44100;
end
if (!isvector(wav))
error("playsound: X must be a vector");
endif
# Write it as a 16-bit signed, little endian (though the amaaazing docs don't say the endianness)
# If it is integers we assume it is 16 bit signed. Otherwise we assume in the range -1:1
if (isfloat(wav))
X = min(max(wav(:), -1), 1) * 32767; # Why matlab & octave do not have a clip() function... I do not know.
else
X = min(max(wav(:), -32767), 32767) + 32767;
endif
unwind_protect
file = tmpnam ();
fid = fopen (file, "wb");
fwrite (fid, X, "int16");
fclose (fid);
# Making aplay (alsa) the default, because let's be honest: it is still way more reliable than
# the mess that is pulseaudio.
if (exist("/usr/bin/aplay") == 2)
system(sprintf("/usr/bin/aplay --format=S16_LE --channels=1 --rate=%d \"%s\"", samplerate, file))
elseif (exist("/usr/bin/paplay") == 2)
system(sprintf("/usr/bin/paplay --format=s16le --channels=1 --rate=%d --raw \"%s\"", samplerate, file))
endif
unwind_protect_cleanup
unlink (file);
end_unwind_protect
endfunction
This is still a very hacky function. But it should be at least a little bit more reliable than playaudio
! I'll leave an implementation of soundsc
as an exercise for the reader.