Skip to content

Comments

Advisoryspeed2.0#1

Open
ssysm wants to merge 14 commits intomasterfrom
advisoryspeed2.0
Open

Advisoryspeed2.0#1
ssysm wants to merge 14 commits intomasterfrom
advisoryspeed2.0

Conversation

@ssysm
Copy link

@ssysm ssysm commented Sep 29, 2024

No description provided.

HaoZhouGT and others added 14 commits September 15, 2024 14:34
as openpilot suggests, remain back compatability
Create the advisory_speed_receiver.py Module

This module will be responsible for reading advisory speed data from the second USB port and publishing it to the OpenPilot messaging system.
add the new python process for receiving the advisory speed
set the added new message name advisorySpeed, ensuring consistency
note that this will allow longitudinal_planner to access 'advisorySpeed' and we can modify the long control there
here the demo usage is to use advisorySpeed as another cruise speed limit
Using UDP (ethernet) connection
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:

  • Convert your PR to a draft unless it's ready to review
  • Read the contributing docs
  • Before marking as "ready for review", ensure:
    • the goal is clearly stated in the description
    • all the tests are passing
    • the change is something we merge
    • include a route or your device' dongle ID if relevant

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.

2 participants