firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
fix the typo in 'persistence' across the file.

fix the typo in 'persistence' across the file.

This function is not checking the level anymore and may be move the functionality at end of fill water end state and transition to next chamber fill state directly,

This function is not checking the level anymore and may be move the functionality at end of fill water end state and transition to next chamber fill state directly,

Check pressure limit.

Check pressure limit.

Does this supposed to be 14 or 17 PSI (based on manifold) for B2.1 hardware?

Does this supposed to be 14 or 17 PSI (based on manifold) for B2.1 hardware?

Is this code updated for B2.1 fill implementation?

Is this code updated for B2.1 fill implementation?

Update Inputs and outputs.

Update Inputs and outputs.

fix the typo.

fix the typo.

declare a local variable to get these function values and use the variables here?

declare a local variable to get these function values and use the variables here?

remove blank space before =

remove blank space before =

Can we split the functions into two files: 1) DryBicart.c/h - Manages drybicart fill, supply, drain & depressurizing states and 2) MixingControl: for acid/bicarb open/close loop functionality?

Can we split the functions into two files: 1) DryBicart.c/h - Manages drybicart fill, supply, drain & depressurizing states and 2) MixingControl: for acid/bicarb open/close loop functionality?

Same as above.

Same as above.

Can we keep getBicarbMixControlInterval( removing drybicart) from the function name?

Can we keep getBicarbMixControlInterval( removing drybicart) from the function name?

Do we need to call updateDryBicartOperation() here to determine the appropriate state.

Do we need to call updateDryBicartOperation() here to determine the appropriate state.

remove blank lines

remove blank lines

Align comment

Align comment

function adjusts the volume of acid and bicarb based on the target conductivity of bicarb and dialysate solution?

function adjusts the volume of acid and bicarb based on the target conductivity of bicarb and dialysate solution?

Put TRUE on left side of ==

Put TRUE on left side of ==

"u" comes before "y".

"u" comes before "y".

Add alarm message.

Add alarm message.

BROADCAST_INTERVAL?

BROADCAST_INTERVAL?

Remove the commented code and add the comment for below function

Remove the commented code and add the comment for below function

As suggested earlier in other code review, split this function into few init functions and call it here.

As suggested earlier in other code review, split this function into few init functions and call it here.

removed

removed

pre-gen does not need substitution rate as its only treatment related.

pre-gen does not need substitution rate as its only treatment related.

its used in handleDialysateDeliveryRequestMsg() to set the sub pump's speed. please review the changes thoroughly.

its used in handleDialysateDeliveryRequestMsg() to set the sub pump's speed. please review the changes thoroughly.

removed

removed

is it a dummy parameter? where are you using it ?

is it a dummy parameter? where are you using it ?

Why pre-gen missing substitutionrate ? not required ?

Why pre-gen missing substitutionrate ? not required ?