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
split pandaState into pandaState and peripheralState #196
Conversation
Not really sure what you're trying to accomplish by splitting these out like this. Wouldn't it suffice to just do something like this:
or even just
where the first entry will always be the internal one? |
The goal is to split driving related tasks from what we hacked into the panda state to control the device. Think this is a good change regardless of multiple panda support. It also makes the upcoming changes to boardd easier to reason about since you can have two different threads and you can pass the appropriate panda to each one. |
ca253b9
to
e4ff64c
Compare
log.capnp
Outdated
@@ -393,6 +387,12 @@ struct PandaState @0xa7649e2575e4591e { | |||
harnessStatus @21 :HarnessStatus; | |||
heartbeatLost @22 :Bool; | |||
|
|||
voltageDEPRECATED @0 :UInt32; |
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.
deprecated fields usually go at the bottom
3e834f1
to
e80c48e
Compare
No description provided.