td-firmware / firmware / App / Modes

ModeTreatment.h

Clone Tools

Stats

Commits this week: 2
Total Committers: 4
Last Commit: 20 Feb
Commits this week: 2

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
Friday 20 Feb
1:42 pm

Raghu Kallala

Updated

Updated

1:36 pm

Raghu Kallala

Updated

Updated

1:35 pm

Sean Nash

I think this function should be in ModeTreatment or Messaging.

I think this function should be in ModeTreatment or Messaging.

1:34 pm

Sean Nash

update params in header.

update params in header.

1:34 pm

Raghu Kallala

you are right, I missed it here. Fixed.

you are right, I missed it here.
Fixed.

1:33 pm

Sean Nash

Move pop to be right after last structure to be packed.

Move pop to be right after last structure to be packed.

1:28 pm

Sean Nash

I thought acid fields were F32?

I thought acid fields were F32?

1:14 pm

Sean Nash

Update inputs/outputs after updating function.

Update inputs/outputs after updating function.

1:14 pm

Sean Nash

We need to look for user request to resume from pause where we would restart BP (at last rate we were at before pause) and re-open art/ven pinch valves and transition back to either ramp or run sta...

We need to look for user request to resume from pause where we would restart BP (at last rate we were at before pause) and re-open art/ven pinch valves and transition back to either ramp or run state (whichever state we were in when we paused).

1:14 pm

Sean Nash

In this state, we still need to signal transition to dialysis when target volume is reached (same as in ramp state). We also need to look for user request to pause (same as in ramp state).

In this state, we still need to signal transition to dialysis when target volume is reached (same as in ramp state).
We also need to look for user request to pause (same as in ramp state).

1:12 pm

Sean Nash

We do need an else if here for when user does change the blood flow rate. In that case, we would want to go to run state where we would run continuously at the requested rate without ramping anymor...

We do need an else if here for when user does change the blood flow rate. In that case, we would want to go to run state where we would run continuously at the requested rate without ramping anymore. Ask Nico which message UI sends when user changes blood flow rate.

And we need another else if for when user asks to pause blood prime and we would stop BP (and maybe close art/ven pinch valves) and go to pause state.

1:10 pm

Sean Nash

Input and Output is cumulativeBloodPrimevolume_mL.

Input and Output is cumulativeBloodPrimevolume_mL.

1:07 pm

Sean Nash

signal function will take us out of blood prime, so no reason to change state here. We would not want to go to run state even if we weren't leaving - run state is for when user changes blood flow r...

signal function will take us out of blood prime, so no reason to change state here. We would not want to go to run state even if we weren't leaving - run state is for when user changes blood flow rate which they have not done here.

1:05 pm

Sean Nash

Raghu is fixing this in his Tx Params branch, but you should fix it too. TUBING_BLOOD_PRIME_VOLUME_ML is not a treatment parameter. See Raghu's fix and duplicate it here.

Raghu is fixing this in his Tx Params branch, but you should fix it too. TUBING_BLOOD_PRIME_VOLUME_ML is not a treatment parameter. See Raghu's fix and duplicate it here.

12:49 pm

Sean Nash

12:49 pm

Vendor - TEL - Varshini Nagabooshanam

LEAHI-TD-FIRMWARE-LDT-3126_Blood Prime - TD
LEAHI-TD-FIRMWARE-LDT-3126_Blood Prime - TD
12:41 pm

Raghu Kallala

removed both enums

removed both enums

11:26 am

Raghu Kallala

Updated

Updated

11:26 am

Raghu Kallala

Updated

Updated

Thursday 19 Feb
7:17 am

Raghu Kallala

Had that question. Thanks Changed to else block

Had that question. Thanks
Changed to else block

7:13 am

Raghu Kallala

Updated

Updated

7:07 am

Raghu Kallala

This will be gone once we have institutional records, do you still want me to move it to a separate func?

This will be gone once we have institutional records, do you still want me to move it to a separate func?

7:06 am

Raghu Kallala

Fixed

Fixed

Wednesday 18 Feb
10:09 pm

Sean Nash

Can we uncomment these and send msg?

Can we uncomment these and send msg?

10:08 pm

Sean Nash

Can we uncomment this and send the limits to the UI?

Can we uncomment this and send the limits to the UI?

10:06 pm

Sean Nash

Should we cast these to (S32)?

Should we cast these to (S32)?

10:04 pm

Sean Nash

These 2 last values have already been used above. Change to 154 and 155.

These 2 last values have already been used above. Change to 154 and 155.

10:02 pm

Sean Nash

Can this stuff down here be done in a separate function that is called here? This function is too big.

Can this stuff down here be done in a separate function that is called here? This function is too big.

10:02 pm

Sean Nash

Should this just be an "else" to above condition?

Should this just be an "else" to above condition?

9:58 pm

Sean Nash

When do you plan on removing this?

When do you plan on removing this?