Nokia melody player

[dmarc1234]

I am trying to create a Nokia melody (old ringtone) player using SineWaveGenerator and StreamCopy. I can get the notes to play but they all roll into a constant playback with no rests between notes.
What I am trying to achieve is to stop the note, silence, after it has played for its duration before playing the next note.

I am setting up the sinewave output as follows:

AudioInfo info(44100, 2, 16);
I2SStream i2s;
SineWaveGenerator<int16_t> sineWave(32000);
GeneratedSoundStream<int16_t> sound(sineWave);
VolumeStream volume(sound);
StreamCopy copier(i2s, volume);

and

// Initialise I2S codec
auto cfg = i2s.defaultConfig(TX_MODE);
cfg.bits_per_sample = 16;
cfg.channels = 1;
cfg.pin_bck = GPIO_I2S_BCLK;
cfg.pin_ws = GPIO_I2S_LRCLK;
cfg.pin_data = GPIO_I2S_DOUT;
i2s.begin(cfg);

// Initialise volume control
auto vol_config = volume.defaultConfig();
vol_config.allow_boost = true;
volume.begin(vol_config);

Then play the note as follows:

sineWave.begin(info, audio_frequency);
    long endTime = millis() + (audio_duration / 2);
    while (millis()<endTime){
      copier.copy();
    }
    sineWave.end();

There is then a delay of 20mS before then next note is played. Is there a way of stopping the audio output or playing silence for the 20mS ?

[pschatzmann]

Why don't you just use the RTTTLOutput class for this ?

You can also use the apprach used by that class which is just setting the frequency to 0 to switch off the tone.
Don't use millis(), but the calculated number of samples to drive your processing: e.g. you could use the StreamCopy copyMs() for this.

[dmarc1234]

I totally miissed that class, I will give it a try, thank you.

[dmarc1234]

Okay, used the example to get it working but Sax was just too buzzy but managed to get the SineWaveGenerator working with it. Only issue I have is changing the output amplitude. I have tried a GeneratedSoundStream and VolumeStream but the RTTTLOutput class will not take the VolumeStream object. How can I chnage the amplitude before playing.

Currently using:

SineWaveGenerator<int16_t> sineWave(32000);
RTTTLOutput<int16_t> rtttl(sineWave, i2s);

and

    sineWave.begin(info);
    rtttl.begin(info);
    rtttl.print(audio_melody);

[pschatzmann]

In the SineWaveGenerator you currently define an amplitude of 32000 which might be a bit loud: why don't you just decrease this value ?

You can also just add a VolumeOutput (or VolumeStream) to the chain:

I2SStream i2s;
VolumeOutput vol(i2s);
SineWaveGenerator<int16_t> sineWave(32000);
RTTTLOutput<int16_t> rtttl(sineWave, vol);

[dmarc1234]

Okay, the VolumeStream on the i2s object worked well I can now change the volume on the fly. One final question is it possible to stop and restart the playback. I have tried undertaking a rtttl.end() and sineWave.end(); before starting things off again, as above, but I just get clicks in time with the duration of the current playback followed by the new playback so it sounds llike the rtttl.end() is not releasing from the play_notes() function in RTTTLOutput.h

[pschatzmann]

Not sure what you mean with stop: If you want to abort playing a longer tune, you would just write out the tune (in the loop) in smaller pieces.

I recommend to call begin() before replaying the same or another tune, though in most cases it might do this automtically.

[dmarc1234]

A mechanism that allows the currently playing tune to stop playback. So what you would recommend is to break the tune down into shorter sections and feed this to rtttl.print() in chunks, seems like a way around it if thats waht you mean.

[dmarc1234]

I have tried breaking the tune down into the first configuration section + one note and then individual notes with each one as a note + comma and rtttl.print() each of these and it does not work, I get the first note and then nothing is played.
I am experimenting with adding a flag and function to abort play to the RTTTLOutput.h code so that I can exit the parse_notes() function but currently I get Vector exceptions when exiting and starting a new rtttl.begin() so something is not quite right.
Reason for all of this is that I am operating the RTTTL playback as a task and the user presses a button on the hardware to select the next tune so they have melody options.
Any pointers in the RTTTLOutput.h changes would be very much appreciated, I am sure I am missing something, probably ring buffer issues.

[pschatzmann]

Maybe the easyest would be to use a separate task for the playback: you can stop the task at any time...

[pschatzmann]

I committed a correction: the following test sketch is now working as expected

#include "AudioTools.h"
#include "AudioTools/AudioLibs/AudioBoardStream.h"
#include "queue"

AudioInfo info(44100, 2, 16);
SineWaveGenerator<int16_t> sineWave(32000);  
AudioBoardStream out(AudioKitEs8388V1);
RTTTLOutput<int16_t> rtttl(sineWave, out);
std::queue<std::string> queue;

// Arduino Setup
void setup(void) {
  // Open Serial
  Serial.begin(115200);
  AudioToolsLogger.begin(Serial, AudioToolsLogLevel::Info);
  delay(5000);

  // start I2S
  Serial.println("starting I2S...");
  auto config = out.defaultConfig(TX_MODE);
  config.copyFrom(info);
  out.begin(config);

  // Setup sine wave (optional)
  sineWave.begin(info);

  // Setup RTTTL queue
  queue.push("ComplexDemo: d=8, o=5, b=140:");
  queue.push("c4 ");
  queue.push("e ");
  queue.push("g ");
  queue.push("c6 ");
  queue.push("a5 ");
  queue.push("g4 ");
  queue.push("e ");
  rtttl.begin(info);
  Serial.println("started...");
}

// Arduino loop - copy sound to out
void loop() {
  if (!queue.empty()) {
    auto& str = queue.front();
    rtttl.print(str.c_str());
    Serial.println(str.c_str());
    queue.pop();
  } else
    delay(1000);
}

[dmarc1234]

Thank you for your help, I will try it out.