anfractuosity / flipperscope

Attempt at an oscilloscope application for the flipper zero
136 stars 3 forks source link

Flipper crashes #1

Closed yesterday4 closed 1 year ago

yesterday4 commented 1 year ago

The flipper crashes, sometimes when measuring, sometimes just after using the "run" function. I don't know if that's important but it crashed many times in a row when measuring 90hz at 1ms. Using it on the unleashed firmware. It is possible to use it, sometimes it doesn't crash for sometime. After rebooting, flipper shows: "MPU fault, possibly stack overflow". Anyway, thank you for making this cool application.

anfractuosity commented 1 year ago

Thanks, I just noticed that after I updated my flipper zero today, I'm not sure if it's something that I've changed in the code here today, or if something has changed in the firmware.

Am looking into it further now.

yesterday4 commented 1 year ago

Thank you.

anfractuosity commented 1 year ago

Are you able to try a new .fap executable?

scope.zip

I've been running for around ~10mins, switching between voltage/frequency mode.

I'm just going to push the changes I made now too, if you're able to compile this repo yourself.

yesterday4 commented 1 year ago

I'm going to try the new .fap executable right now. I didn't really understand what you said in your last sentence :). I'm trying to learn how to compile applications for flipper.

yesterday4 commented 1 year ago

The new .fap works great. The application never crashes now. Thank you for fixing the issue so quickly. Only the 0.1s and 1s time scales don't really work for me. They either show nothing at all or just one frame of a distorted wave.

anfractuosity commented 1 year ago

Cool, thanks for reporting back.

Regarding the 1s period, it means the analogue to digital converter takes a sample every 1 seconds in this application.

The buffer size is 128 samples though and currently you only see a change in the output when that is full, so after 128 seconds.

I'll investigate if I can make it so you don't need to wait for the whole buffer to be filled, before the display is updated.

anfractuosity commented 1 year ago

I've created a new issue regarding improving the displaying of waveforms (see #2)

Would it be ok to close this issue now?

yesterday4 commented 1 year ago

Understood. Yes, thank you.

anfractuosity commented 1 year ago

Thanks again for reporting. Feel free to create issues for feature requests/other improvements.