Problem with Axon And LV-MaxSonar-EZ0

[Hanoush]

When I use this function "sonar_MaxSonar(a2dConvert8bit(0))"    in sensors.c. It gives me weird output that isn't reasonable at all & I don't even know if it outputs in meters or inches or what ?

and when I add delay in "my function" or inside "while(1)" ,it gives me this message "calledbadISR" in the hyperterminal with the rest of the Sonar values. I am not using Interrupts and I tried every way to change my code but it kept on showing up. any idea what is this ?

[pomprocker]

Would you mind posting some code for me?

[Hanoush]

Code: [Select]


long unsigned int MaxVal=0;
void control(void)
{
while(1)
{

if(button_pressed())
{
delay_ms(50);
MaxVal=sonar_MaxSonar(a2dConvert8bit(0));
rprintf("MaxVal=%d\r\n",MaxVal);

}
delay_ms(1000);
}
This is the last code,it reads the value from the Analog pin only when I push the button on my Axon. After getting the value I want to change it to meters but I don't know how. this is for my graduation project and it's my first robot in my life and I have to do a flying robot in 3 months 

[Admin]

it gives me this message "calledbadISR" in the hyperterminal with the rest of the Sonar values

hmmmm another person recently got this error, appears to be a new bug with the latest software release . . .

Anyway, this code:
sonar_MaxSonar(a2dConvert8bit(0))

returns an int value. Try changing:
long unsigned int MaxVal=0;

to:
int MaxVal=0;

and let us know if that fixes it.

After getting the value I want to change it to meters but I don't know how.

That function automatically returns a value in inches. Just multiply it by 2.54 and you'll get centimeters.

[Tomas]

it gives me this message "calledbadISR" in the hyperterminal with the rest of the Sonar values

hmmmm another person recently got this error, appears to be a new bug with the latest software release . . .

Anyway, this code:
sonar_MaxSonar(a2dConvert8bit(0))

returns an int value. Try changing:
long unsigned int MaxVal=0;

to:
int MaxVal=0;

and let us know if that fixes it.

After getting the value I want to change it to meters but I don't know how.

That function automatically returns a value in inches. Just multiply it by 2.54 and you'll get centimeters.

Isnt it divide by 2,54?

And it the problem continues, heres what solved calledbadISR for me. Simply downgrading to 1.01. Im not 100% sure, but I believe I havent had that error since I downgraded(and that I didnt have that problem before I upgraded). And btw, if you are installing a new version, follow the tutorial! I found that just downloading the source code and opening the project created problems for me which continued until I did it the right way.

[Admin]

Isnt it divide by 2,54?

nope, multiply



edit: So what exactly do you guys add to the code that triggers this badISR? Are you guys using interrupts?

[Hanoush]

I tried to use int instead of long unsigned int but it is the same error. The error appear when there is delay,I think it doesn't have time to appear when I remove the delay because the code keep on executing so fast. anyway,Thanks a lot for you help. I will try to downgrade and install the source code again.

[Admin]

The error appear when there is delay,I think it doesn't have time to appear when I remove the delay because the code keep on executing so fast.

If I can narrow the problem down I can squash this bug. When you use this code there is no error?

Code: [Select]

long unsigned int MaxVal=0;
void control(void)
{
while(1)
{

if(button_pressed())
{
//delay_ms(50);//delay is removed

MaxVal=sonar_MaxSonar(a2dConvert8bit(0));
rprintf("MaxVal=%d\r\n",MaxVal);

}
delay_ms(1000);
}

[Hanoush]

Code: [Select]

long unsigned int MaxVal=0;
void control(void)
{
while(1)
{

if(button_pressed())
{
//delay_ms(50);//delay is removed

MaxVal=sonar_MaxSonar(a2dConvert8bit(0));
rprintf("MaxVal=%d\r\n",MaxVal);

}
//delay_ms(1000);  you have to remove this also
}
you have to remove both delays. I wanted to ask about another thing,how to make the MaxSonar more accurate ?

[Hanoush]

Thanks Tomas,The error disappeared once I downgraded.

[Tomas]

Dont worry about it.

And to Admin, I will try using 1.05 again (and I will try reproducing the error) when Im back in schedual. Right now Im one week behind on my project because of the problems But I will fault search it in a few days.

[Admin]

The main advantage of 1.05 is that is has PWM. If you don't plan to use PWM, just uses the older software for now.

I wanted to ask about another thing,how to make the MaxSonar more accurate ?

Well, you'd want to use I2C, or at least 10 bit adc and not convert to cm:

MaxVal=a2dConvert10bit(0);

When you convert to cm, you get rounding errors, you don't actually need to convert for the robot to work.

[yerbie]

and when I add delay in "my function" or inside "while(1)" ,it gives me this message "calledbadISR" in the hyperterminal with the rest of the Sonar values. I am not using Interrupts and I tried every way to change my code but it kept on showing up. any idea what is this ?

the reason for the badISR printing out is because there was an uncaught interrupt.  This seems to mean that somehow the delay function is causing some interrupt to be fired that you weren't expecting.  I'll try out your code to see if I can get the same results.  Having the uncaught interrupt is bad because it can cause the axon to spontaneously reboot.

This quote is from the gcc-avr interrupt page: http://www.gnu.org/savannah-checkouts/non-gnu/avr-libc/user-manual/group__avr__interrupts.html

If an unexpected interrupt occurs (interrupt is enabled and no handler is installed, which usually indicates a bug), then the default action is to reset the device by jumping to the reset vector.

[Hanoush]

Well I will be using PWM for my motors, can't I take the PWM code from the new Source code and put in the old one ?

[Tomas]

Well I will be using PWM for my motors, can't I take the PWM code from the new Source code and put in the old one ?

If this is the code you are using, then it works in 1.01 too:

http://www.societyofrobots.com/robotforum/index.php?topic=5590.0

[Hanoush]

I will try it out as soon as I get my motor driver. Thanks a lot