toneAC v1.2 - Twice the volume, higher quality, higher frequency, etc.

First of all, I love the library. It will be my first choice in most applications.
I hope to test the Uno/Mega/Leonardo/ATmega8 with version 1.2, after that it should be ready.
That single line of code makes some of my tests sound a lot better.

I have made a checklist for version 1.2, perhaps there are one to two things that make sense.

(1) The code increased a little (just a little) which is always a problem for the ATmega8
library + example version with 1.0, Arduino Uno : 1516
library + example version with 1.1, Arduino Uno : 1670
library + example version with 1.0, ATmega8 : 1336
library + example version with 1.1, ATmega8 : 1472

(2) The ISR uses millis(), so a roll-over of the millis is possible. The length is no longer fail-safe, so using the delay() function with noToneAC() is preferrable for critical situations.

(3) You don't use the 'L' for long values. I don't mind about it in the sketch, but you could use it in your library. I prefer to use 0L, 256L, - 1L and so on, when long integers are involved.

(4) The examples have the extension *.pde. I would prefer *.ino

(5) In the example you use '/* .... */' for comment, but everwhere else '//'.

(6) In the example, you use a potmeter. But for a small test with the library, a very quick result (showcase) would be needed. And a potmeter will make it unnecessary complicated.

(7) You made 10 volume steps, which is great. But for a 'ping' sound, a smooth decrease of the volume is needed. Or am I asking too much for the library ? Are there open source examples of tunes with volume ?

( 8 ) Why is the sound different for this example ?

int i, freq, vol;

for (i = 0; i<5; i++)
{  
  freq = 900;
  for (vol=10; vol>0; vol--)
    toneAC( freq , vol, 60, false);
  freq = 540;
  for (vol=10; vol>0; vol--)
    toneAC( freq, vol, 60, false);
  delay( 3000);
}

for (i = 0; i<5; i++)
{  
  freq = 900;
  for (vol=10; vol>0; vol--)
  {
    toneAC( freq , vol, 0, true);
    delay(60);
  }
  freq = 540;
  for (vol=10; vol>0; vol--)
  {
    toneAC( freq, vol, 0, true);
    delay(60);
  }
  noToneAC();
  delay( 3000);
}

Krodal:
(1) The code increased a little (just a little) which is always a problem for the ATmega8

Great pains have been taken to keep the code as small as possible. I'm a huge believer in running as much as possible in the background so around 150 additional bytes I believe are important. It's still much smaller than the tone library. I show the Arduino Uno code to be 1604 bytes with Arduino v1.0.2 not 1670. And with the ATmega8 I have it as 1398 instead of 1472 (again v1.0.2). As a test I created a toneACmini() function that only has a frequency and duration (automatically runs in the background). With toneACmini() instead of toneAC() it went down to 1488 bytes on the Uno. But, not sure if 116 bytes is worth confusing people.

Krodal:
(2) The ISR uses millis(), so a roll-over of the millis is possible. The length is no longer fail-safe, so using the delay() function with noToneAC() is preferrable for critical situations.

It uses unsigned long int, so it's about a 50 day range. That should be good for almost any situation. The rollover of micros is critical, but ms hardly ever happens in the real world. Although, I am working on a project right now that I'm trying to get it to run virtually forever.

Krodal:
(3) You don't use the 'L' for long values. I don't mind about it in the sketch, but you could use it in your library. I prefer to use 0L, 256L, - 1L and so on, when long integers are involved.

If there's an advantage I'm not aware of let me know. Most of my longs are unsigned longs. Not sure if that makes a difference.

Krodal:
(4) The examples have the extension *.pde. I would prefer *.ino

They need to be .ino for a library so old versions of Arduino can find them. I'm running only v1.0.2 so everything is a .pde. I change them to .ino just for the library. Other library creators told me to use .ino

Krodal:
(5) In the example you use '/* .... */' for comment, but everwhere else '//'.

I hardly ever use '/* .... */'. I use it in that particular case to try and identify that you can do a bunch of stuff there, not just one line.

Krodal:
(6) In the example, you use a potmeter. But for a small test with the library, a very quick result (showcase) would be needed. And a potmeter will make it unnecessary complicated.

The library is designed for sound, not LEDs. The sample is "toneAC_demo" which doesn't use a pot. The one you're talking about is an advanced one that I don't mention anywhere as it's only for more advanced users that may want to use toneAC for other purposes.

Krodal:
(7) You made 10 volume steps, which is great. But for a 'ping' sound, a smooth decrease of the volume is needed. Or am I asking too much for the library ? Are there open source examples of tunes with volume ?

You're asking too much. I got out my SPL meter and it was HARD to get even 10 volume levels. A square wave just doesn't work well for creating a true volume level. I had to cheat to get even 10, there was really only 9 but that would just be confusing.

Krodal:
( 8 ) Why is the sound different for this example ?

They're very close. But, the first loop is inferior because it uses the foreground method and therefore there's a short period of time where there's no sound, producing a non-constant sound and clicking. This way is designed for a laymen who who isn't too comfortable with programming but they can still generate sound using this method and would probably be totally happy with that. The second way is a more proper way that a higher level programmer would use, knowing that the first way would have a very slight but still noticeable gap between notes reducing quality. I would only ever use the second loop, as I did with the original tone() library as well.

The idea that something runs in the background is very foreign to many novice programmers. They love delay statements, or a command that just does something for as long as they specify and then returns to them when it's done. So, toneAC makes it easy for the masses, but still powerful for those who better understand an interrupt/timer driven paradigm.

Tim

My sketch size was different, because I used my own example.

I like the mini function. Perhaps it will not be used often, it's nice to have as an extra.

Using 'L' or 'UL' for long numbers is something I do for myself, so that I know what I'm doing.

I agree with everything else. Perhaps that ping-pong sound (the second smooth one) can be used as an example ? I'm trying to make more sounds that show what the volume can do.

A page in the playground section would be nice. I can make one, with links to your code and this thread.

But, not sure if 116 bytes is worth confusing people.

think not, but by the time their sketches fill up all memory and the last 116 bytes are critical I assume they aren't confused any more by a stripped lib :wink:

robtillaart:

But, not sure if 116 bytes is worth confusing people.

think not, but by the time their sketches fill up all memory and the last 116 bytes are critical I assume they aren't confused any more by a stripped lib :wink:

If I only knew how to set a define in a sketch so the library knew about it I'd be done by now.

Tim

Don't bother about the mini, it's not that important.

Example 1 : alert (soft and loud)

  // soft and loud alert.
  // For the toneAC library.
  // public domain
  int i, j;

  // Soft alert
  for( i=0; i<10; i++)
  {
    for( j=0; j<3; j++)
    {
      toneAC( 720, 3, 0, true);
      delay( 50);
      toneAC( 610, 3, 0, true);
      delay( 50);
      noToneAC();
      delay( 30);
    }
    delay( 500);
  }

  delay( 2000);
  
  // Loud alert
  for( i=0; i<10; i++)
  {
    for( j=0; j<3; j++)
    {
      toneAC( 720, 10, 0, true);
      delay( 50);
      toneAC( 610, 10, 0, true);
      delay( 50);
      noToneAC();
      delay( 30);
    }
    delay( 500);
  }

  delay( 2000);

Krodal:
Don't bother about the mini, it's not that important.

Example 1 : alert (soft and loud)

  // soft and loud alert.

// For the toneAC library.
  // public domain
  int i, j;

// Soft alert
  for( i=0; i<10; i++)
  {
    for( j=0; j<3; j++)
    {
      toneAC( 720, 3, 0, true);
      delay( 50);
      toneAC( 610, 3, 0, true);
      delay( 50);
      noToneAC();
      delay( 30);
    }
    delay( 500);
  }

delay( 2000);
 
  // Loud alert
  for( i=0; i<10; i++)
  {
    for( j=0; j<3; j++)
    {
      toneAC( 720, 10, 0, true);
      delay( 50);
      toneAC( 610, 10, 0, true);
      delay( 50);
      noToneAC();
      delay( 30);
    }
    delay( 500);
  }

delay( 2000);

If you want a loud alert from a piezo, it's best to first find its harmonic frequency. You can do it with a miltimeter. Just look for the frequency where the most current is drawn. It will typically be quite a large spike right at the loudest point. If you want something loud, this will be the frequency to play. With a piezo, it's typically 4,000 Hz.

Tim

Example 2 : A whistling kettle with boiling water sound.
This is a full sketch.

// Example for the toneAC library
// A whistling kettle with boiling water sound.
// public domain

#include <toneAC.h>

void setup()
{
}

void loop()
{
  int i, freq, freq_extra, freq_shift, vol;

  freq = 1440;
  vol = 1;

  for (i=0; i<1000; i++)
  {
    freq_shift = random (-15, 15);
    freq += freq_shift;
    
    // Stay around the frequency 1440
    if (freq < 1440)
      freq ++;
    else
      freq --;

    vol = i/50;      
    if (vol > 10)
      vol = 10;
    if (i > 800)
      freq_extra = (vol * vol) + 800;
    else
      freq_extra = (vol * vol) + i;
    
    toneAC( freq + freq_extra, vol, 0, true);
    delay(30);
  }
  noToneAC();
  
  delay(5000);
}

Krodal:
Don't bother about the mini, it's not that important.

To save a bit of compiled code size, use toneAC(). instead of noToneAC(). noToneAC() was meant to be familiar to the Tone library and the noTone() command. But, it's totally not needed for toneAC() and as a bonus you save 10 bytes.

Also, I created a TONEAC_TINY switch that allows you to save around 110 bytes that uses an alternate version of toneAC(). Would be useful for projects where maximum speed or the lowest code size is required.

I'm considering creating another library named something like ToneTiny or NewTone where it's a plug-in replacement for Tone including pin assignment but is much smaller, faster, and better quality. I know lots of people that have timer interrupt issues with the Tone library and are looking for an alternative. Not that toneAC doesn't provide that already, but toneAC is designed more for highly accurate AC switching speed and maximum volume.

Tim

teckel:
I'm considering creating another library named something like ToneTiny or NewTone where it's a plug-in replacement for Tone including pin assignment but is much smaller, faster, and better quality. I know lots of people that have timer interrupt issues with the Tone library and are looking for an alternative. Not that toneAC doesn't provide that already, but toneAC is designed more for highly accurate AC switching speed and maximum volume.

It was so easy I just did it, NewTone is a working library. Plug-in replacement for the Tone library, but over 1,200 bytes smaller compiled code size, faster, better quality sound, etc. Also uses timer 1 if you're having a conflict on timer 2 with the Tone library and some other library or hardware. Seriously took 30 minutes to write.

A little bit surprising is that toneAC still produces slightly smaller code (60-170 bytes). This is possible because toneAC doesn't need to find the ports and masks for the pins as it's fixed. But, I believe NewTone still has a place for those who need smaller, tighter code, yet need the flexibility of connecting to any pin. toneAC is still better in every way, with the exception being the fixed pins, but that's also why it's so good.

I'll probably package up NewTone and upload it later today for those who need a flexible drop-in replacement for the Tone library.

Tim

I'm considering creating another library named something like ToneTiny or NewTone

TinyTony? sounds Italian, would fit the Arduino :wink:

Example 3: Tone Tutorial with fade out tones.
You have to listen to this for yourself. It is not like a piano, because it sounds very computer/robotic.

Use the tone tutorial : http://arduino.cc/en/Tutorial/tone
But connect the loudspeaker to two output pins as is required for the toneAC library.

Replace the setup() function with this:

// Example for toneAC with tone fade out.
// Using the tone tutorial : http://arduino.cc/en/Tutorial/Tone
// public domain

void setup()
{
  for (int thisNote = 0; thisNote < 8; thisNote++) {
    // to calculate the note duration, take one second 
    // divided by the note type.
    //e.g. quarter note = 1000 / 4, eighth note = 1000/8, etc.
    int noteDuration = 1000/noteDurations[thisNote];
    
    // Add some to the length for the pauses between the notes.
    noteDuration += noteDuration/2;
    int i, vol;
    // Setting the volume to 12 is a trick to reduce the code.
    // The maximum volume is 10, but 12 is allowed.
    vol = 12;
    for (i=0; i<=noteDuration; i+=25)
    {
      // Double the frequency to make it sound better.
      toneAC( 2* melody[thisNote], vol, 0, true);
      // fade out
      if (vol > 0)
        vol--;
      delay( 25);
    }
    noToneAC();
  }
}

@teckel

I tried your sketch on a Bobuino with a 1284 chip, the speaker buzzed but nothing more. The board is found halfway down this page. Cross Roads Electronics

This may be asking too much but, would you consider making your library work with the 1284P? I have built a few boards using the chip.

This page may help you in the process if your interested. Arduino on ATmega1284P | maniacbug

I will be happy to test for you of course. :slight_smile:

cyclegadget:
I tried your sketch on a Bobuino with a 1284 chip, the speaker buzzed but nothing more. The board is found halfway down this page. Cross Roads Electronics

This may be asking too much but, would you consider making your library work with the 1284P? I have built a few boards using the chip.

This page may help you in the process if your interested. Arduino on ATmega1284P | maniacbug

I will be happy to test for you of course. :slight_smile:

I had believed it would work with the 1284P. Don't have one to confirm, but looking into it further I see that the ATmega1284P is different. I've updated my development v1.2 to support ATmega640, ATmega644, ATmega1281, ATmega1284P and ATmega2561. I've private messaged you a file for you to test to verify it works on the ATmega1284P. The pins you should use are D12 and D13 (the timer 1 PWM pins) on that microcontroller.

Let me know how it works.

Tim

Krodal:
Example 2 : A whistling kettle with boiling water sound.

I don't have the code handy - but waaaay back when I was kid, I remember playing around on my TRS-80 Color Computer with the PLAY command; there was a way, via using certain values (I think random notes played with very, very short durations) with a rapidly decreasing volume level (the CoCo had a 6 bit DAC - so there were 64 volume levels) - you could make an "explosion" or "gun" sound of sort...

...care to try making something like that?

:wink:

Since the toneAC library is ideal for kids toys (extra loud, no nasty clicks, volume control), these kind of sounds came to my mind. But so far I could not make something that sounds right.

Krodal:
Since the toneAC library is ideal for kids toys (extra loud, no nasty clicks, volume control), these kind of sounds came to my mind. But so far I could not make something that sounds right.

If you'd like to post your sound effect sketches to the toneAC library page I believe I can arrange it. You need a Gmail email address and let me know what it is and I'll try to set it up for you. Never done it before, but it seems like it's possible.

Tim

No need to set it up for me. I hope to add one or two more sounds with the library, and I will post them in this thread. I don't know if others want to contribute, and I also can't find a site with melodies and sounds that make use of the volume.

Example 4: Random sound

This is just a random. To make it random to the human ear, the frequency is not linear.
It can be used as startup-sound or to simulate that device is totally confused.
Set the volume to your own need.

// Random sound
// public domain

int i, j, frequency, volume, duration;

for( i=0; i<1000; i++)
{
  j = random (5, 60);
  frequency = j * j;
  volume = random(6, 11);
  toneAC( frequency, volume, 0, true);
  j = random( 1, 12);
  duration = j * j;
  delay( duration);
}
noToneAC();
  
delay(2000);

Example 5 : misuse of the toneAC library

The next example is able to generate noise sounds.
Both random functions are part of the delay. The random change in volume is needed for the noise.

// example of noise sounds with toneAC library
// public domain
//
// The lower and higher frequency define the sound.
//     300, 1500 : noise
//     340, 345   : computer noise
//     600, 14500 : rain drops noise

#define FREQUENCY_LOWER   300
#define FREQUENCY_HIGHER   1500
  
int i, frequency, volume;

for( i=0; i<10000; i++)
{
  volume = random(6, 11);
  frequency = random( FREQUENCY_LOWER, FREQUENCY_HIGHER);
  toneAC( frequency, volume, 0, true);
  delay(1);
}
noToneAC();
  
delay(2000);

First of all this library is awesome, and I'm very pleased that I found it. Registered just for this.

Secondly though, I'm running into trouble because of the 9 & 10 pin requirements. (Arduino Uno)

Is there a way someone could point in the right direction to modify the library to use pins 5 & 6? (or other PWM pins in general?) Or perhaps I misunderstand and the relevant timer is ONLY accessible on 9 & 10.

Any reason we couldn't use Timer 0 or Timer 2, per: Arduino Playground - TimerPWMCheatsheet

Alternately, I think it'd be an excellent addition to the library to be able to set options to switch between all possible PWM pins, just to give a little bit more flexibility.