•  

Comment Results

Review Name Created Custom Fields Content
HD-DEN-759-1 03 Dec 2019

What is the difference between UI is communicating and UI did communicate?

DIALIN-1 06 Dec 2019

RESOLVED.

HD-DEN-759-1 04 Dec 2019

RESOLVED in CODE WALKTHROUGH.

DG-DEN-1125-1 13 Dec 2019

I do have a document folder where I check for doxygen output locally

HD-DEN-759-1 04 Dec 2019

I would suggest using && operator to reduce complexity of code

UI-DEN-1090-1 30 Dec 2019

Isn't there a library or a method that already does this?

HD-DEN-431-1 30 Dec 2019

If Peman means to have "if ( function() != 0 )" I totally agree with him and I don't know why it should be in reverse?

DIALIN-1 10 Dec 2019

Message structure defined in our Messages spreadsheet and eventually in our SDDs.

DIALIN-1 13 Dec 2019

Done

DG-DEN-1125-1 11 Dec 2019

Done

DIALIN-1 16 Dec 2019

RESOLVED.

DIALIN-1 26 Nov 2019

How is the message Timeout!!!! related to received_message being equal to None?

DG-DEN-1125-1 12 Dec 2019

Peman, please read e-mail. The changes cannot be seen here because issues we are having. You need to see changes in bitbucket.

DIALIN-1 11 Dec 2019

Extra line

DIALIN-1 16 Dec 2019

RESOLVED.

LEAHI-APPLICATION-LDT-1021-1 24 Jun 2025

Updated and added keyword

DIALIN-1 16 Dec 2019

RESOLVED.

UI-DEN-1090-1 19 Dec 2019

RESOLVED.

UI-DEN-1090-1 19 Dec 2019

Same here.

DG-DEN-1125-1 16 Dec 2019

Because this will be a common file. I don't think it will be worth to go through that exercise. If this was unique with DG, I would agree with the comment.

LEAHI-SW-LDT-2071-1 29 Aug 2025

RESOLVED

DIALIN-1 17 Dec 2019

RESOLVED

DIALIN-1 05 Dec 2019

Please Change the Cargo variable name to payload...

DG-DEN-1125-1 17 Dec 2019

Remove OFF Button message.

HD-DEN-431-1 06 Jan 2020

Done!

UI-DEN-1090-1 19 Dec 2019

Some Constants are one time use and doesn't add any value to the code to collect them in one place.

UI-DEN-1090-1 19 Dec 2019

Fixed

UI-DEN-1090-1 19 Dec 2019

This code is for test to be able to enable or disable CanInterface , MessageDispatcher console out to be able to debug without rebuild even on target.
And I mentioned at top of the code that it's a Test code for debuging

HD-DEN-1312-1 06 Jan 2020

Needed for re-purposing a SPI4 pin for GPIO (used by dialysate outlet pump).

UI-DEN-1090-1 19 Dec 2019

Same here, These are aligned parameters as I mentioned to better identify the code bugs.

HD-DEN-431-1 06 Jan 2020

Software alarms will be added.

HD-DEN-431-1 06 Jan 2020

Added #defines for the years

HD-DEN-431-1 06 Jan 2020

Done!

DG-DEN-1125-1 20 Dec 2019

Stop Button removed because none on DG.

DG-DEN-1125-1 07 Jan 2020

RESOLVED in CODE WALKTHROUGH.

DG-DEN-1125-1 13 Dec 2019

"{" should be on separate line (throughout code).

HD-DEN-431-1 07 Jan 2020

RESOLVED in CODE WALKTHROUGH.

HD-DEN-431-1 30 Dec 2019

There are macros in Common.h for getting high and low bytes of a word.

HD-DEN-431-1 07 Jan 2020

RESOLVED in CODE WALKTHROUGH.

HD-DEN-431-1 30 Dec 2019

length should be range checked.

HD-DEN-431-1 07 Jan 2020

RESOLVED in CODE WALKTHROUGH.

HD-DEN-431-1 30 Dec 2019

I understood the usage, thanks.

HD-DEN-759-1 15 Nov 2019

RESOLVED in CODE WALKTHROUGH.

LEAHI-DD-FIRMWARE-LDT-2198-1 11 Sep 2025

removed

HD-DEN-759-1 15 Nov 2019

Explicitly explain in words, 0, 1, 2, ...

HD-DEN-759-1 03 Dec 2019

uiDidCommunicate is latched. It is useful to know that UI came up and was communicating at some time. uiIsCommunicating is cleared when the getter function is called so the UI has to keep communicating to set the flag again and again (like a comm watchdog).

UI-DEN-608-1 03 Dec 2019

RESOLVED.

HD-DEN-431-1 30 Dec 2019

Very good point does it build if the definition is not present at the function definition ?

HD-DEN-759-1 26 Nov 2019

Is this Diality file or TI file?

HD-DEN-759-1 26 Nov 2019

RESOLVED in CODE WALKTHROUGH.