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

Sending messages is broken #8

Open
jonofmac opened this issue Feb 1, 2022 · 0 comments
Open

Sending messages is broken #8

jonofmac opened this issue Feb 1, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@jonofmac
Copy link
Owner

jonofmac commented Feb 1, 2022

Currently vpw_analyzer does not have the ability to send messages.

This is primarily due to the fact that the ELM327 command set does not allow a graceful way to send a message without risking losing a message incoming.

The basic process to send a message while datalogging with a ELM327 device is this:

  1. Configure device
  2. Enter AT MA (monitor all) mode to see all bus traffic (vpw_analyzer's primary mode)
  3. Cancel AT MA mode
  4. Configure desired headers
  5. Send message
  6. Device waits some amount of time to see if it gets a response to the specified headers (ignoring other traffic)
  7. Enter AT MA mode to continue seeing messages

Steps 3-6 mean that there's a window of time that messages can (and most probably will) be dropped. This is an inherent weakness with the standard ELM327 commands.

Some more advanced devices (OBDX VT Pro) offer additional modes to allow transmission while receiving messages. The decision needs to be made to support both modes or only the safer OBDX modes.

@jonofmac jonofmac added the enhancement New feature or request label Feb 1, 2022
@jonofmac jonofmac added this to the 1.0 milestone Feb 1, 2022
@jonofmac jonofmac self-assigned this Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant