substitution_pump.py

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
I am happy to see that this old code is improving. I would recommend using the Conversion module instead of strings and DataTypes, as it handles everything automatically. Examples can be found in t...

I am happy to see that this old code is improving.
I would recommend using the Conversion module instead of strings and DataTypes, as it handles everything automatically.
Examples can be found in the following file in this CR:
https://devapps.diality.us/cru/#LEAHI-DIALIN-LDT-4484-1CFR-84688

Please add a docstring for the function. including the intention and parameters with their intentions.

Please add a docstring for the function.
including the intention and parameters with their intentions.

Same for CanChannels. The Channels we are referring to here are Leahi's or Denali's. Channels mainly have no meaning beyond their priorities when a collision occurs.

Same for CanChannels.
The Channels we are referring to here are Leahi's or Denali's.
Channels mainly have no meaning beyond their priorities when a collision occurs.

I recommend DenaliMessage or LeahiMessage, as CanMessage is generic, whereas Leahi/Denali indicate that the protocol is also implemented in the message we are sending. I'm not sure why the name was...

I recommend DenaliMessage or LeahiMessage, as CanMessage is generic, whereas Leahi/Denali indicate that the protocol is also implemented in the message we are sending.
I'm not sure why the name was changed, as it is a lot of change.
Also, there is no CAN Message.
There is a CAN Frame and a Leahi/Denali Message.

updated

updated

Remove extra spaces before comment

Remove extra spaces before comment

fixed. thanks!

fixed. thanks!

LEAHI-DIALIN-LDT-4484_[Dialin] Create Dry Self Test module in dialin
LEAHI-DIALIN-LDT-4484_[Dialin] Create Dry Self Test module in dialin
LDT-4001: added pump id to sensor list in message handler

LDT-4001: fixed naming of data structures

  1. … 1 more file in changeset.
LDT-4001: updates to sub pump based on refactoring that was merged into staging.

  1. … 3 more files in changeset.
I would like to ask to check out the substitution pump on the develop branch and update it based on that. There are some changes thanks to the merging of the refactor.

I would like to ask to check out the substitution pump on the develop branch and update it based on that.
There are some changes thanks to the merging of the refactor.

looks there are some more payloads in publish function. please add the other parameters here.

looks there are some more payloads in publish function. please add the other parameters here.

LDT-4000: updated message IDs. Updated test config for beta 1.9 and 2.0. implemented substitution pump module

    • -0
    • +112
    ./substitution_pump.py
  1. … 2 more files in changeset.
LEAHI-DIALIN-LDT-4001_HDF - FW Implementation - 2/4:
LEAHI-DIALIN-LDT-4001_HDF - FW Implementation - 2/4: