Testers wanted for large buffer crash fix #237
Closed
PedroKTFC
announced in
Announcements
Replies: 2 comments 2 replies
-
|
Compiles and seems to be working from initial tests. I will report back if issues arise - note that previous versions have been stable for me without any crashing/re-booting. Thank you for all your time and effort spent on this project - it's a joy to be able to interact with the car locally. |
Beta Was this translation helpful? Give feedback.
2 replies
-
|
just installed this version, is running ok at the moment. Will test and report back if any issues. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I've been working on how to fix the app from crashing when the message buffer is extended too far. Too far is different for different users. Unfortunately, I haven't been able to track down the root cause so I've concentrated on reducing the use of dynamic memory. I've converted many of the app's uses of dynamic memory to use static (fixed) memory. This might make it a little larger but won't make it use any more memory.
I ran a version of this yesterday for several hours uninterrupted and the version that's in repair (both use a 512 byte buffer, as compared with just over 300 bytes in the default branch). I'd appreciate some tests of this with different boards if anyone has time.
This branch also has a change to reduce the number of data requests made to the car when it's awake. Rather than poll everything every time, some things, such as tyre pressures which never change, are now polled less frequently:
They should be read on an initial connect (eg when it reconnects ot the car comes home).
Currently, these are fixed in the code, would people prefer them to be configurable?
To use this version, you need to change your yaml (you may need to clean build files or even clean all):
Add the following substitutions:
and use this for the packages section (if you have the old version with separate lines for each package, just delete them and replace with this version):
Many thanks in advance.
Beta Was this translation helpful? Give feedback.
All reactions