ESP32 core panic

[lodesmets]

Hey, I tried your code, but it kept crashing. After some research it seems that the problem is with _dir.

According to this, you aren't allowed to do float arithmetic in an interupt.
void WeatherMeters::timer() gets called on interrupt. And void WeatherMeters::timer() calls float WeatherMeters::adcToDir(uint16_t value)

[pilotak]

Thanks for finding a bug, according to the issue the easiest fix is to make _dir double instead of float. Can you please test it? perhaps submitting a PR?

[lodesmets]

I tested it, and did a pull request both here and in pilotak/MovingAverageAngle#2 (that one also uses floats)

Thanks btw for the great library

[pilotak]

ok i didn't realise this will also affect the MovingAverageAngle library (perfect investigation 👍) which i want to keep with floats due to FPU which leads me to an idea - I can easilly adjust the example for timers to bring the timer() function to main loop which would solve the problem. What do you think?

[lodesmets]

That is indeed the way I have done it now