Clone Tools
  • last updated a few minutes 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

Merge branch 'LDT-4721-extra_param' into staging

  1. … 1 more file in changeset.
LEAHI-DIALIN-LDT-4160_[Request] Get function for full detailed Alarm Table
LEAHI-DIALIN-LDT-4160_[Request] Get function for full detailed Alarm Table
Fixed non working modules

  1. … 2 more files in changeset.
fixes

  1. … 6 more files in changeset.
Rename to CanChannels

Rename to CanChannels

Rename to CanChannels

Rename to CanChannels

Let it be there. Because of this, i could not complete the 30+ data broadcast.

Let it be there. Because of this, i could not complete the 30+ data broadcast.

align the indent or remove blank space before =

align the indent or remove blank space before =

remove blank space before =

remove blank space before =

LEAHI-DIALIN-LDT-4721_Add extra parameter to start pregen request
LEAHI-DIALIN-LDT-4721_Add extra parameter to start pregen request
Put this into a dataclass folder under common, so we start structuring it.

Put this into a dataclass folder under common, so we start structuring it.

Not in use, don't need to add

Not in use, don't need to add

2 empty lines please

2 empty lines please

2 lines between functions please

2 lines between functions please

remove empty line

remove empty line

remove 3rd empty line

remove 3rd empty line

Rename to CanChannels

Rename to CanChannels