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

Added reserved fields for body controls #537

Merged
merged 6 commits into from
Sep 27, 2023
Merged

Conversation

mitchellgoffpc
Copy link
Contributor

No description provided.

log.capnp Outdated
@@ -2265,6 +2265,10 @@ struct Event {
livestreamWideRoadEncodeData @121 :EncodeData;
livestreamDriverEncodeData @122 :EncodeData;

customReservedText0 @124 :Text;
Copy link
Contributor

Choose a reason for hiding this comment

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

would consider changing the text to data

@mitchellgoffpc mitchellgoffpc merged commit b1a1afe into master Sep 27, 2023
16 checks passed
@mitchellgoffpc mitchellgoffpc deleted the body-controls branch September 27, 2023 23:38
sunnyhaibin added a commit to sunnypilot/msgq that referenced this pull request Oct 13, 2023
* pre-commit: autoupdate hooks (commaai#532)

Co-authored-by: adeebshihadeh <[email protected]>

* Update README.md (commaai#533)

* pre-commit: autoupdate hooks (commaai#535)

Update pre-commit hook versions

Co-authored-by: adeebshihadeh <[email protected]>

* services: capitalize constant service list (commaai#534)

* rename services

* keep more unique name

* type this

* Bump pandaStates to 10hz (commaai#536)

* 10 Hz

* constant

* fix

* revert

* fix peripheralState freq

* revert peripheral

* Bringing sunnypilot

* Refactor service list and deprecate certain fields

Refactored service list naming convention in messaging/init.py and services.py for consistency, changing `service_list` to `SERVICE_LIST`. Additionally, fields as `laneWidth`, `lProb`, `rProb`, `dProb` in `LateralPlan` struct and `brakeLights` in `CarEvent.EventName` have been marked as DEPRECATED as part of an ongoing effort for codebase clean-up. Changes also include updating the frequency of `pandaStates` service and commenting out a debug statement in msgq.cc, enhancing overall system performance.

* pre-commit: autoupdate hooks (commaai#538)

Update pre-commit hook versions

Co-authored-by: adeebshihadeh <[email protected]>

* Added reserved fields for body controls (commaai#537)

* Added bodyReserved0-2 text fields

* Added bodyReserved0 to services list

* add more bodyReserved to servcies py

* bodyReserved -> customReservedText

* :Text -> :Data

---------

Co-authored-by: Kacper Rączy <[email protected]>

* zmq: assert the same PID (commaai#540)

Enforce the same PID in ZMQ pub sockets

* pre-commit: autoupdate hooks (commaai#541)

Update pre-commit hook versions

Co-authored-by: adeebshihadeh <[email protected]>

* Bringing sunnypilot

* Refactor service list and deprecate certain fields

Refactored service list naming convention in messaging/init.py and services.py for consistency, changing `service_list` to `SERVICE_LIST`. Additionally, fields as `laneWidth`, `lProb`, `rProb`, `dProb` in `LateralPlan` struct and `brakeLights` in `CarEvent.EventName` have been marked as DEPRECATED as part of an ongoing effort for codebase clean-up. Changes also include updating the frequency of `pandaStates` service and commenting out a debug statement in msgq.cc, enhancing overall system performance.

* Bringing sunnypilot's cereal changes aligned

* Converging cereal changes

* accidentally duplicated some literals

* Update log.capnp

Co-authored-by: Jason Wen <[email protected]>

---------

Co-authored-by: Adeeb Shihadeh <[email protected]>
Co-authored-by: adeebshihadeh <[email protected]>
Co-authored-by: JJ <[email protected]>
Co-authored-by: Shane Smiskol <[email protected]>
Co-authored-by: Miguel Fernandez <[email protected]>
Co-authored-by: Mitchell Goff <[email protected]>
Co-authored-by: Kacper Rączy <[email protected]>
Co-authored-by: Jason Wen <[email protected]>
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.

3 participants