-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added 'Supported Message Types' section to README #44
Conversation
Dockerfile Optimization
Merge candidate_r1 into master
Sync Develop Branch with Master for Q1 2024 Release Changes
Update dockerhub.yml
…ssage types to README
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Problem
While the PPM mentions basic safety messages in the README, it does not provide an explicit list of message types that are supported. This may result in users attempting to use the PPM to process message types other than BSMs.
Solution
The 'Supported Message Types' section has been added to the README, which makes it very clear that only BSMs are supported at this time.
Dependent PR
The
docs/supported-message-types
branch was created usingsync/pull-usdot-changes-into-develop
as a base, the PR for which should be merged first: #43