dd-firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Is this temporary? If so, put a TODO to remove later.

Is this temporary? If so, put a TODO to remove later.

Remove extra spaces between { and msg name.

Remove extra spaces between { and msg name.

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC e...

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC exec).

Do these belong in the spent chamber fill group?

Do these belong in the spent chamber fill group?

Add space before ).

Add space before ).

Add space between if and (.

Add space between if and (.

Be consistent on whether you have a blank line between break and next case.

Be consistent on whether you have a blank line between break and next case.

Keep blank line here.

Keep blank line here.

What are these coefficients for? At least describe in comments, if not the names.

What are these coefficients for? At least describe in comments, if not the names.

Remove extra blank line.

Remove extra blank line.

Added.

Added.

I think you are seeing previous commit. It is not there in the present code.

I think you are seeing previous commit. It is not there in the present code.

Done. Thank you.

Done. Thank you.

Changed. Thank you.

Changed. Thank you.

We still need this if statement.

We still need this if statement.

Add a 1 second persistence on this alarm

Add a 1 second persistence on this alarm

Handle transition here after changing the flush logic transitionToGenPermeateDefeaturedMode(); requestGenWaterDefStart();

Handle transition here after changing the flush logic

transitionToGenPermeateDefeaturedMode();
requestGenWaterDefStart();

Need to remove this handler

Need to remove this handler

Add blank line before return statement

Add blank line before return statement

Remove handlePreGenPDefInletWaterState

Remove handlePreGenPDefInletWaterState

Update FP_Pre_GenP_Def_Mode_States enum in FPDefs.h

Update FP_Pre_GenP_Def_Mode_States enum in FPDefs.h

Call this in both Pre Gen: Verify Water and Gen Permeate

Call this in both Pre Gen: Verify Water and Gen Permeate

Greater than or equal to for low limit pressureP13 >= MAX_INLET_RO_PUMP_PRESSURE_WARNING_LOW_PSIG Also give space before and after parentheses.

Greater than or equal to for low limit
pressureP13 >= MAX_INLET_RO_PUMP_PRESSURE_WARNING_LOW_PSIG

Also give space before and after parentheses.

Put quadratic and linear terms in parentheses.

Put quadratic and linear terms in parentheses.

is it merge issue ?

is it merge issue ?

If it is not used for serial message, its OK, but if you are opting for packing , club it together , in line number 36 , or bring that structure defined on top to here

If it is not used for serial message, its OK, but if you are opting for packing , club it together , in line number 36 , or bring that structure defined on top to here