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

Support for odx-c and odx-cs file formats #81

Closed
premkumar-masilamani opened this issue Oct 12, 2022 · 3 comments
Closed

Support for odx-c and odx-cs file formats #81

premkumar-masilamani opened this issue Oct 12, 2022 · 3 comments

Comments

@premkumar-masilamani
Copy link
Contributor

premkumar-masilamani commented Oct 12, 2022

Hi @andlaus - We have a requirement to parse the communication parameters from PDX files (odx-c and odx-cs) to be able to configure the diagnostic clients. I would like to know, if this file format support is in the road map. Please let us know.

If not planned already, we would be happy to add the functionality to this library. Please let us know if there are any concerns.

Thanks,
Prem

@andlaus
Copy link
Collaborator

andlaus commented Oct 12, 2022

I intend to implement support for parsing comparam fragments eventually, but there are no concrete plans at the moment. (it was just too low in my priorities list.) If you want to implement this feature, I'm more than happy to help out a bit :)

@dmholtz
Copy link
Contributor

dmholtz commented Dec 17, 2022

Having merged PR #101 #98 #92 and possibly some more I wonder if this issue is still valid?

@andlaus
Copy link
Collaborator

andlaus commented Dec 17, 2022

you're right: comparam subsets are fully internalized by now. closing.

@andlaus andlaus closed this as completed Dec 17, 2022
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

No branches or pull requests

3 participants