xiphonics / picoTracker

BSD 3-Clause "New" or "Revised" License
213 stars 18 forks source link

stop resetting rp2040 on project load #161

Open maks opened 7 months ago

maks commented 7 months ago

Now that we have ram usage under much better control, it would be good to stop doing a reset on project load. Or if we aren't ready to commit to that, it would be nice to make it a compile time option, as I've found it doesn't play well with a open USB connection when the reset happens, causing a connected linux laptop to put the serial connection into a bad state that prevents applications from reopening the serial port device until the picotracker is unplugged and then replugged in.

democloid commented 7 months ago

Well it's not about having memory under control, is about memory leaks. Unless we find the source of the leak, each time we go back to project screen we will leak more memory and eventually have problems. We are close to eliminating that issue due to other things I'm working on tho (moving all memory to static allocations and also starting on project loaded mode rather than on the project screen)

maks commented 7 months ago

Well it's not about having memory under control, is about memory leaks.

Sorry that's what I meant, with most dynamic allocs now replaced with static there shouldn't be many places where dynamic mem could be leaking.