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.

LDt-4733: fixed typo in voltage module

LDT-4733: voltage monitoring updates.

  1. … 1 more file in changeset.
LEAHI-DIALIN-LDT-4733_(Request) Add variables to capture AC/DC heater electrical data
LEAHI-DIALIN-LDT-4733_(Request) Add variables to capture AC/DC heater electrical data
Update the name.

Update the name.

updated

updated

Remove extra spaces before comment

Remove extra spaces before comment

If it's mixing control, then rename the parameters to reflect it, these names suggest a different module

If it's mixing control, then rename the parameters to reflect it, these names suggest a different module

Fix Spacing please to be consistent with the rest of the variables

Fix Spacing please to be consistent with the rest of the variables

Please maintain the FW-SW Integration.xlsx for MSG_ID reservation. Meaning, add these there too to avoid overwriting or deleting them

Please maintain the FW-SW Integration.xlsx for MSG_ID reservation.
Meaning, add these there too to avoid overwriting or deleting them

LEAHI-DIALIN-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
LEAHI-DIALIN-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
LDT-4485: updates to msg id list. creation of version and calibration request commands

  1. … 1 more file in changeset.
LDT-4612 mixing control braodcast param plus dry bicart testconfig updates

  1. … 1 more file in changeset.
LEAHI-DIALIN-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
LEAHI-DIALIN-LDT-4612_Dialysate Composition (Dry Bicarbonate) - FW Iteration - 4/1:
Merge branch 'LDT-4721-extra_param' into staging

  1. … 1 more file in changeset.
LDT-4612 addressed review comments