dd-firmware / firmware / App / Controllers

DryBiCart.h

Clone Tools

Stats

Commits this week: 1
Total Committers: 4
Last Commit: 05 Jun
Commits this week: 1

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 09 Jun
4:05 pm

Vinayakam Mani

fix the typo in 'persistence' across the file.

fix the typo in 'persistence' across the file.

4:00 pm

Vinayakam Mani

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,

3:56 pm

Vinayakam Mani

Check pressure limit.

Check pressure limit.

3:50 pm

Vinayakam Mani

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?

3:45 pm

Vinayakam Mani

Is this code updated for B2.1 fill implementation?

Is this code updated for B2.1 fill implementation?

3:33 pm

Vinayakam Mani

Update Inputs and outputs.

Update Inputs and outputs.

3:29 pm

Vinayakam Mani

fix the typo.

fix the typo.

3:19 pm

Vinayakam Mani

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?

3:18 pm

Vinayakam Mani

remove blank space before =

remove blank space before =

3:15 pm

Vinayakam Mani

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?

3:11 pm

Vinayakam Mani

Same as above.

Same as above.

3:10 pm

Vinayakam Mani

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

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

2:54 pm

Vinayakam Mani

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

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

2:29 pm

Vinayakam Mani

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?

2:20 pm

Vinayakam Mani

Add alarm message.

Add alarm message.

2:18 pm

Vinayakam Mani

Remove the commented code and add the comment for below function

Remove the commented code and add the comment for below function

2:14 pm

Vinayakam Mani

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.

12:06 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-4183_Dialysate Composition (Dry Bicarbonate) - FW Implementation - 2/1:
LEAHI-DD-FIRMWARE-LDT-4183_Dialysate Composition (Dry Bicarbonate) - FW Implementation - 2/1:
Monday 01 Jun
12:05 pm

Vinayakam Mani

All these are more of debug related overrides (acid and bicarb P & I gain). you can keep them in one function, when the implementation is complete, you may remove these. The init function is too bi...

All these are more of debug related overrides (acid and bicarb P & I gain). you can keep them in one function, when the implementation is complete, you may remove these. The init function is too big, group them and split in the functions and call here.

12:03 pm

Vinayakam Mani

Some of these parameters can be changed over td proxy commands, need not to implement additional overrides.

Some of these parameters can be changed over td proxy commands, need not to implement additional overrides.

12:00 pm

Vinayakam Mani

I guess all these MACRO implementations should move to function/s to compute the final acid and bicarb volume, that would help to calculate the volumes when Na and Bicarb params changed during trea...

I guess all these MACRO implementations should move to function/s to compute the final acid and bicarb volume, that would help to calculate the volumes when Na and Bicarb params changed during treatment.

11:52 am

Vinayakam Mani

looks Bicarb and sodium settings interchanged? There are multiple macros on the same values. remove the duplicates.

looks Bicarb and sodium settings interchanged? There are multiple macros on the same values. remove the duplicates.

11:48 am

Vinayakam Mani

33.0?

33.0?

11:47 am

Vinayakam Mani

Add comment for each macro.

Add comment for each macro.

11:41 am

Vinayakam Mani

For all floats add F at the end, across the file.

For all floats add F at the end, across the file.

11:40 am

Vinayakam Mani

Align the comment

Align the comment

Friday 22 May
11:11 am

Vendor - TEL - Jashwant Gantyada

10:50 am

Sean Nash and Vinayakam Mani

7:51 am

Vinayakam Mani

Thursday 21 May
3:24 pm

Vendor - TEL - Jashwant Gantyada

restored

restored