Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for CAN logging #342

Merged
merged 25 commits into from
May 29, 2024
Merged

Add support for CAN logging #342

merged 25 commits into from
May 29, 2024

Conversation

LegacyNsfw
Copy link
Owner

@LegacyNsfw LegacyNsfw commented Dec 4, 2023

I have been doing a bunch of data logging in my Corvette lately, so that's gotten me motivated to improve the logger.

There are a couple of UI changes that I think most people will appreciate. First, the "start/stop recording" button is much bigger, so it's easier to press it with your thumb while you're driving. :) Second, all of the parameters are now always displayed on the left side of the window.

And, since I have a couple of CAN devices in my Corvette, so I added support for CAN logging. Currently, only one CAN interface is supported. But, it's cheap:

https://www.seeedstudio.com/USB-CAN-Analyzer-p-2888.html

If you need AFR in your logs, I recommend AEM's X-Series gauges, which support support CAN, so you don't need to wire them into your PCM.

There are also a few new log parameters for the 7603 operating system, but I have to admit that not all of them are reliable.

There are also a couple of tiny improvments to PCM Hammer. The long pause before "Kernel upload 48%" always makes my heart skip a beat, so now it prints "Permission to upload granted" before it start the upload. Also, it logs the VPW messages for OBDX devices.

The next things on my to-do list - which probably won't happen in this order, but which will probably happen:

  • a more dashboard-like UI for log parameters, so they'll be easier to read at a distance
  • enable/disable log saving via the Air Conditioning switch, or something else that doesn't require reaching for the laptop
  • more log parameters for the 7603 operating system (the idle PID terms aren't right yet...)

@LegacyNsfw LegacyNsfw marked this pull request as ready for review May 3, 2024 04:01
@LegacyNsfw LegacyNsfw merged commit 45232a2 into develop May 29, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant