Skip to content
This repository has been archived by the owner on Apr 1, 2024. It is now read-only.

EDS parser bugs #87

Open
ryanpdx opened this issue Nov 9, 2023 · 0 comments
Open

EDS parser bugs #87

ryanpdx opened this issue Nov 9, 2023 · 0 comments
Assignees
@venomboss985
Labels
bug Something isn't working

Comments

@ryanpdx
Copy link
Member

ryanpdx commented Nov 9, 2023

The EDS parser does not handle the following:

  • Hex values for NodeId do not work
  • The misspelling of DeviceComissioning (that is how it is spelled in the spec) is not supported. I would suggest supporting both DeviceCommisioning and DeviceComissioning
  • All keys should be case insensitive; e.g.: NODEID=, NodeId=, and NodeID=, should all work the same.

Steps to reproduce the behavior:

  • Start program with attached dcf and send a IMU heartbeat with cansend: $ cansend vcan0 738#05

imu.dcf
@ryanpdx ryanpdx added the bug Something isn't working label Nov 9, 2023
@venomboss985 venomboss985 mentioned this issue Jan 4, 2024
Closed
@venomboss985 venomboss985 self-assigned this Jan 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@venomboss985 venomboss985

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ryanpdx @venomboss985