td-firmware / firmware / App / Services

DDInterface.c

Clone Tools

Stats

Commits this week: 3
Total Committers: 4
Last Commit: 28 Apr
Commits this week: 3

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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Today
16 mins ago

Michael Garthwaite and Vinayakam Mani

27 mins ago

Sean Nash

I think DD (and TD) want mL/min and UI wants L/hr. TxParams should have already calculated this rate (in mL/min) and should probably have a get function you can call from here to access it. When se...

I think DD (and TD) want mL/min and UI wants L/hr. TxParams should have already calculated this rate (in mL/min) and should probably have a get function you can call from here to access it. When sending UF rate to UI, we would convert back to L/hr first, but everywhere else we would work with mL/min.

37 mins ago

Sean Nash

Remove extra blank line.

Remove extra blank line.

37 mins ago

Sean Nash

Remove extra space before function name.

Remove extra space before function name.

38 mins ago

Sean Nash

This is not a Dialin message, so move up above all of these "test" functions.

This is not a Dialin message, so move up above all of these "test" functions.

38 mins ago

Sean Nash

Why is this in TDCommon? It should probably be in MessagePayloads.h or DDDefs.h so both DD and TD f/w can share this payload structure. And the common definitions should be added to appropriate dox...

Why is this in TDCommon? It should probably be in MessagePayloads.h or DDDefs.h so both DD and TD f/w can share this payload structure. And the common definitions should be added to appropriate doxygen group.

44 mins ago

Raghu Kallala

As the fw haven't requested the gen dialysate it is expected to be in pre-gen mode

As the fw haven't requested the gen dialysate it is expected to be in pre-gen mode

1 hr ago

Michael Garthwaite

1 hr ago

Vinayakam Mani

Do we have a return confirmation that the command is accepted by DD?

Do we have a return confirmation that the command is accepted by DD?

1 hr ago

Vinayakam Mani

Is the UF rate unit in ml/min (instead of hr.)?

Is the UF rate unit in ml/min (instead of hr.)?

2 hrs ago

Vinayakam Mani

Are we assuming the sub mode return is pre-gen sub mode always?

Are we assuming the sub mode return is pre-gen sub mode always?

2 hrs ago

Michael Garthwaite

create the getDialGoodToDeliverStatus() function and have that return true rather than have the assignment here

create the getDialGoodToDeliverStatus() function and have that return true rather than have the assignment here

3 hrs ago

Michael Garthwaite

comment still needed?

comment still needed?

Wednesday 29 Apr
2:01 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-4188_SW/FW integration to run a Treatment from UI
LEAHI-TD-FIRMWARE-LDT-4188_SW/FW integration to run a Treatment from UI
10:57 am

Raghu Kallala

10:44 am

Vinayakam Mani

Tuesday 28 Apr
6:13 pm

Sean Nash

2:31 pm

Sean Nash

???

???

2:30 pm

Sean Nash

Do we need a parameter for confirmation?

Do we need a parameter for confirmation?

Friday 17 Apr
7:33 am

Raghu Kallala

Added to be consistent. Once institutional records are implemented will store it in there instead of a macro

Added to be consistent. Once institutional records are implemented will store it in there instead of a macro

Wednesday 15 Apr
12:02 pm

Vinayakam Mani

DO we need bicarb conversion factor as well?

DO we need bicarb conversion factor as well?

9:46 am

Sean Nash

preweight can be non-zero

preweight can be non-zero

Tuesday 14 Apr
1:34 pm

Sean Nash

Add blank line after declarations.

Add blank line after declarations.

Wednesday 08 Apr
10:39 am

Raghu Kallala

From what I see in UI, it is just a range value not an enum/index

From what I see in UI, it is just a range value not an enum/index

Monday 06 Apr
2:12 pm

Sean Nash

1:37 pm

Sean Nash

Is it an enum/index like Denali then?

Is it an enum/index like Denali then?

12:43 pm

Raghu Kallala

No we are not getting conv factor from UI on it I will have to confirm with Nick to see if we will get the bicarbConvFactor

No we are not getting conv factor from UI on it
I will have to confirm with Nick to see if we will get the bicarbConvFactor

11:01 am

Sean Nash

Align comments.

Align comments.

10:57 am

Sean Nash

Can we just remove this?

Can we just remove this?

10:57 am

Sean Nash

Can we just remove this?

Can we just remove this?