td-firmware / firmware / App / Modes

StateTxDialysis.c

Clone Tools

Stats

Commits this week: 3
Total Committers: 5
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
Tuesday 28 Apr
6:13 pm

Sean Nash and Vinayakam Mani

2:31 pm

Sean Nash

???

???

2:30 pm

Sean Nash

Do we need a parameter for confirmation?

Do we need a parameter for confirmation?

Sunday 26 Apr
1:08 pm

Sean Nash

We can set bolus volume here now. These functions should be moved to TxParams where all of these settings exist and can be updated.

We can set bolus volume here now. These functions should be moved to TxParams where all of these settings exist and can be updated.

1:06 pm

Sean Nash

Why does confirm msg have duration again? If UI resends duration, we have to validate it again. If UI just sends confirmation of previously sent duration, we don't have to validate it again.

Why does confirm msg have duration again? If UI resends duration, we have to validate it again. If UI just sends confirmation of previously sent duration, we don't have to validate it again.

1:02 pm

Sean Nash

These are already in TxParams I believe.

These are already in TxParams I believe.

1:01 pm

Sean Nash

Should these be moved to Services\TxParams?

Should these be moved to Services\TxParams?

12:59 pm

Sean Nash

Change 1 to TRUE

Change 1 to TRUE

12:58 pm

Sean Nash

Agree with name change, but I think BOOL was the correct type. And BOOL is equivalent to U32 anyway so compiler won't care.

Agree with name change, but I think BOOL was the correct type. And BOOL is equivalent to U32 anyway so compiler won't care.

12:56 pm

Sean Nash

I think these message handlers for editing treatment parameters should be moved to Services\TxParams.

I think these message handlers for editing treatment parameters should be moved to Services\TxParams.

12:55 pm

Sean Nash

These should probably be moved to Services\TxParams.h because Messaging.c does not use them.

These should probably be moved to Services\TxParams.h because Messaging.c does not use them.

12:48 pm

Sean Nash

See if we have already defined this stuff in Services\TxParams. TxParams may also have get function(s) for min/max values that you can use.

See if we have already defined this stuff in Services\TxParams. TxParams may also have get function(s) for min/max values that you can use.

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

Thursday 16 Apr
7:32 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-3970_Edit Treatment Parameters During Treatment - FW Implementation - 1/4:
LEAHI-TD-FIRMWARE-LDT-3970_Edit Treatment Parameters During Treatment - FW Implementation - 1/4:
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?

10:56 am

Sean Nash

F suffix.

F suffix.

10:52 am

Sean Nash

Is bicarbonate also a conversion factor?

Is bicarbonate also a conversion factor?

10:49 am

Sean Nash

Can we just remove the old enum now?

Can we just remove the old enum now?

10:16 am

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-473_Treatment Parameters (Create Rx) - TD - 04: DEV - Feature Implementation
LEAHI-TD-FIRMWARE-LDT-473_Treatment Parameters (Create Rx) - TD - 04: DEV - Feature Implementation
10:04 am

Raghu Kallala

Wednesday 25 Feb
9:19 am

Sean Nash