4 Apr
2023
4 Apr
'23
7:13 a.m.
Hi all, I was after some feedback on https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pull/859 which moves the ISOTP polling into what is currently the ISOTP response thread.
This is part of a larger rework I have been doing to add flexibility to the loop and to move the poll-loop operations behind a clean interface.
I'm interested in feedback on the mechanism in the p/r directly, but I'm also happy to provide the usage examples I have (which would be the follow-up commits for Ioniq 5).
I'm also interested in other people's thoughts on where they see what improvements might be had to the poll-list mechanism. Already Done:
- Clean up the blocking poll mechanism
- Allow for multiple concurrent poll lists (still executed sequentially)
- Add once-off non-blocking poll mechanism
- Add a poll-list that allows repeating more-frequently than the primary poll
Ideas:
- Add a user-configurable poller that could possibly utilise the DBC framework (on top of the existing vehicle framework) to allow exploring/experimenting with both passive data as well as polled data. (Given that the DBC framework already does the heavy lifting of interpreting data - adding a poll request to that seems to be a nice way of going)
- ??? Have a poll thread per bus ???
//.ichael