hdfirmware / firmware / App

Modes

Clone Tools

Stats

Commits this week: 0
Total Committers: 15
Last Commit: 19 Nov 25
Commits this week: 0

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 22 Oct 2024
4:59 pm

Dara Navaei

The state machine that is in charge of receiving the response back from the sensor has a timeout and sets the command response as ready.

The state machine that is in charge of receiving the response back from the sensor has a timeout and sets the command response as ready.

1:26 pm

Dara Navaei

Done

Done

1:15 pm

Sean Nash

So now it's enough if 1 or 2 samples in the drift range is enough to re-zero? Still seems too brief to me.

So now it's enough if 1 or 2 samples in the drift range is enough to re-zero? Still seems too brief to me.

1:13 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:13 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:13 pm

Sean Nash

This call to signal pressure stabilization should probably be moved into the request zero function.

This call to signal pressure stabilization should probably be moved into the request zero function.

1:05 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:56 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:51 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:50 pm

Sean Nash

Do we need a else if timeout check here? Can we get stuck waiting for response indefinitely?

Do we need a else if timeout check here? Can we get stuck waiting for response indefinitely?

12:40 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:30 am

Dara Navaei

Removed the static pointer.

Removed the static pointer.

10:29 am

Dara Navaei

Removed the state.

Removed the state.

9:44 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

9:08 am

Dara Navaei

This code has been removed, please check the handler functions. Also, ternary is a one-liner so sometimes it reduces the lines of code.

This code has been removed, please check the handler functions. Also, ternary is a one-liner so sometimes it reduces the lines of code.

9:05 am

Sean Nash

I'm not seeing that. And why isn't the exec returning a state that you assign to result?

I'm not seeing that. And why isn't the exec returning a state that you assign to result?

9:03 am

Sean Nash

Why are you treating bloodLeakZeroingStatus as a pointer here?

Why are you treating bloodLeakZeroingStatus as a pointer here?

9:00 am

Sean Nash

Not clear to me that this is any better than bypassing the dialyzer in previous state and is likely worse. The way you have it now, the dialysate being pushed by DPi is going to run into a stopped ...

Not clear to me that this is any better than bypassing the dialyzer in previous state and is likely worse. The way you have it now, the dialysate being pushed by DPi is going to run into a stopped DPo and cause a big pressure spike and push dialysate into blood side.

8:49 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

8:47 am

Sean Nash

That doesn't answer my question.

That doesn't answer my question.

8:46 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

8:41 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Monday 21 Oct 2024
5:42 pm

Dara Navaei

Just pushed.

Just pushed.

5:40 pm

Dara Navaei

Please check the complete handler.

Please check the complete handler.

5:37 pm

Dara Navaei

I added a static pointer to change the state inside one the exec handlers.

I added a static pointer to change the state inside one the exec handlers.

5:31 pm

Dara Navaei

Added handlers for the state machine.

Added handlers for the state machine.

5:27 pm

Dara Navaei

Done

Done

4:46 pm

Dara Navaei

The zeroBloodLeak() only tries up to 3 calls and then alarms.

The zeroBloodLeak() only tries up to 3 calls and then alarms.

1:43 pm

Dara Navaei

I understand your comment. I am saying stabilizing when the pumps are running might be harder as you stop in other places. I will match to saline state.

I understand your comment. I am saying stabilizing when the pumps are running might be harder as you stop in other places. I will match to saline state.

1:39 pm

Sean Nash

What does that have to do with my comment? Stopping the pumps will also change the pressure. Doesn't help. I'm asking you to look at and understand how the saline bolus works and follow that model ...

What does that have to do with my comment? Stopping the pumps will also change the pressure. Doesn't help. I'm asking you to look at and understand how the saline bolus works and follow that model more closely. I think you are missing things that we need to deal with.