Dashboard

tip: use the logger and set the statement to error level and you can see it in runtime

tip: use the logger and set the statement to error level and you can see it in runtime

I don't see a TODO. If we intend to uncomment this line out later, we should add a TODO comment to the right as a reminder. If we do not intend to uncomment this later, we should just delete it.

I don't see a TODO. If we intend to uncomment this line out later, we should add a TODO comment to the right as a reminder. If we do not intend to uncomment this later, we should just delete it.

Fix alignment of "=".

Fix alignment of "=".

Add a param line in this header.

Add a param line in this header.

Can we delete this commented out line?

Can we delete this commented out line?

Add a params line in function header.

Add a params line in function header.

Why did we change this spelling? Now it's wrong.

Why did we change this spelling? Now it's wrong.

Can we delete this commented out line?

Can we delete this commented out line?

Keep this blank line.

Keep this blank line.

Should be before ModeStandby.

Should be before ModeStandby.

I see we are doing this same logic in multiple places. Can we just have a function that returns the minimum speed for the given dialysate pump?

I see we are doing this same logic in multiple places. Can we just have a function that returns the minimum speed for the given dialysate pump?

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Should assign a local variable to this equation and then return the variable. Can this equation result in a negative number? If so, we should handle that first before converting to unsigned int. Al...

Should assign a local variable to this equation and then return the variable.
Can this equation result in a negative number? If so, we should handle that first before converting to unsigned int.
Also, do we want to do any rounding when converting float to integer?

Remove extra blank line.

Remove extra blank line.

Bamboo Commit: Updated MAlarmMapping.cpp

    • -14
    • +20
    /sources/model/hd/alarm/MAlarmMapping.cpp
Merge branch 'LDT-3600-new_td_testconfig' into develop

Merge branch 'LDT-3629-fp_dd_valve_states_enum_fix' into develop

    • -1
    • +1
    /leahi_dialin/fp/modules/temperatures.py
pressure limit will be unsigned int not signed int with LDT-473..beta CR Add a TODO to update once the 473 is ready to merge

pressure limit will be unsigned int not signed int with LDT-473..beta CR
Add a TODO to update once the 473 is ready to merge

Are these print statements add for debugging? If not, is this allowed

Are these print statements add for debugging?
If not, is this allowed

typo, PUMP_ON_STATE

typo, PUMP_ON_STATE

The TODO has already been mentioned in staging. They have it commented out for strike and hold behavior.

The TODO has already been mentioned in staging. They have it commented out for strike and hold behavior.

LDT-3603 addressed review comments and added missing code

    • -0
    • +62
    /firmware/App/Modes/FPModes/FPModeFault.c
    • -1
    • +4
    /firmware/App/Modes/FPModes/FPModeFault.h
    • -0
    • +52
    /firmware/App/Modes/FPModes/FPModeStandby.c
    • -88
    • +103
    /firmware/App/Modes/FPModes/FPOperationModes.c
    • -48
    • +55
    /firmware/App/Modes/FPModes/ModeGenPermeate.c
    • -2
    • +2
    /firmware/App/Modes/FPModes/ModeGenPermeate.h
That's a question for me, will that be required? Fault has sub modes(3), Standby has 1 sub mode(I am adding code for this as this might be required) Init don't have any and service is not defined yet

That's a question for me, will that be required?

Fault has sub modes(3), Standby has 1 sub mode(I am adding code for this as this might be required)
Init don't have any and service is not defined yet

Updated

Updated

Updated

Updated

Changed and left them as static, as we don't need these outside this unit

Changed and left them as static, as we don't need these outside this unit