td-firmware / firmware / App

Modes

Clone Tools

Stats

Commits this week: 36
Total Committers: 15
Last Commit: 18:32
Commits this week: 36

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
Today
2 hrs ago

Vendor - TEL - Varshini Nagabooshanam

LDT-4127: New changes in blood prime

Options
    • -82
    • +137
    ./StateTxBloodPrime.c
10 hrs ago

Sean Nash

Better to say != TRUE here.

Better to say != TRUE here.

10 hrs ago

Sean Nash

Add TODO comment as to whether we should trigger an alarm/fault here.

Add TODO comment as to whether we should trigger an alarm/fault here.

10 hrs ago

Sean Nash

Add blank line after declaration. Fix indent.

Add blank line after declaration.
Fix indent.

10 hrs ago

Sean Nash

We can un-comment syringe pump stop now.

We can un-comment syringe pump stop now.

10 hrs ago

Sean Nash

I think this function should be private (called by public message handler).

I think this function should be private (called by public message handler).

10 hrs ago

Sean Nash

Should this be static (private)?

Should this be static (private)?

Friday 21 Aug
5:58 pm

Sean Nash

I don't have a good option. If we're going to change it to pull approach later, can't we just ignore rate changes when doing fluid bolus with sub fluid until you implement the pull approach? Are we...

I don't have a good option. If we're going to change it to pull approach later, can't we just ignore rate changes when doing fluid bolus with sub fluid until you implement the pull approach? Are we even able to do sub fluid bolus yet? Do we have HDF tube sets?

5:48 pm

Vendor - TEL - Praneeth Bunne

Sure

Sure

5:44 pm

Vendor - TEL - Praneeth Bunne

I'll implement the pull approach later, this is temporary fix. We need to run Sub pump at Qb rate, if Qb is updated, how can we increase Qsub?

I'll implement the pull approach later, this is temporary fix.
We need to run Sub pump at Qb rate, if Qb is updated, how can we increase Qsub?

5:37 pm

Sean Nash

Should we check to see if blood sitting timer has not exceeded time limit before we resume (similar to what we do from rinseback stop state before we start an additional)?

Should we check to see if blood sitting timer has not exceeded time limit before we resume (similar to what we do from rinseback stop state before we start an additional)?

5:30 pm

Sean Nash

So we will command BP every 50ms for saline (okay, but not the pull approach we talked about) and send DD Qsub commands every 50ms for sub fluid (probably not a good idea)?

So we will command BP every 50ms for saline (okay, but not the pull approach we talked about) and send DD Qsub commands every 50ms for sub fluid (probably not a good idea)?

9:47 am

Michael Garthwaite

Thursday 20 Aug
5:10 pm

Sean Nash

Can we combine this state with prior state? Wait for DD response and mode change in same state?

Can we combine this state with prior state? Wait for DD response and mode change in same state?

5:08 pm

Sean Nash

Only need second condition I think.

Only need second condition I think.

5:02 pm

Sean Nash

This should be switch on requested selection and, in each case, we set next state to wait for disinfect state and set the appropriate command (if not skip) or set next state to wait for treatment (...

This should be switch on requested selection and, in each case, we set next state to wait for disinfect state and set the appropriate command (if not skip) or set next state to wait for treatment (if skip).

4:58 pm

Sean Nash

I think this should be more general start disinfection function that takes a specific disinfection command enum as a parameter.

I think this should be more general start disinfection function that takes a specific disinfection command enum as a parameter.

4:56 pm

Sean Nash

Shouldn't this be UI REQUEST message? Why would we need to receive/handle a message that we send?

Shouldn't this be UI REQUEST message? Why would we need to receive/handle a message that we send?

4:49 pm

Sean Nash

Do we need to send UI a response? Accept/reject?

Do we need to send UI a response? Accept/reject?

4:49 pm

Sean Nash

Is this a message handler? If so, parameter should be a message and option should be extracted from message payload.

Is this a message handler? If so, parameter should be a message and option should be extracted from message payload.

4:48 pm

Sean Nash

Does this command cause DD to transition to standby? It looks like the command to give when we transition into post treatment mode to tell DD to transition to post-gen dialysate mode.

Does this command cause DD to transition to standby? It looks like the command to give when we transition into post treatment mode to tell DD to transition to post-gen dialysate mode.

4:46 pm

Sean Nash

Switch not needed here. Just signal standby mode with the received selection.

Switch not needed here. Just signal standby mode with the received selection.

4:44 pm

Sean Nash

Should be disinfection selection type, not command type. Also, move this down into if statement scope.

Should be disinfection selection type, not command type.
Also, move this down into if statement scope.

4:43 pm

Sean Nash

Why are we signaling a command to standby mode? We should signal the selection that we received from UI. Standby mode (later when we get there) will then look at the selection and decide there whic...

Why are we signaling a command to standby mode? We should signal the selection that we received from UI. Standby mode (later when we get there) will then look at the selection and decide there which command to send to DD.

4:40 pm

Sean Nash

Remove blank line.

Remove blank line.

4:37 pm

Sean Nash

Won't this happen when we transition to auto-eject state? Why do it here?

Won't this happen when we transition to auto-eject state? Why do it here?

4:37 pm

Sean Nash

Shouldn't auto-eject state handle this BP homing?

Shouldn't auto-eject state handle this BP homing?

4:36 pm

Sean Nash

Why are we worried about "handled"? Once confirmed, should we just go to auto eject state?

Why are we worried about "handled"? Once confirmed, should we just go to auto eject state?

4:36 pm

Sean Nash

Why is this if statement separated from the one above? Why not just do this stuff in that if statement?

Why is this if statement separated from the one above? Why not just do this stuff in that if statement?

4:32 pm

Sean Nash

There should be function(s) in DDInterface to tell DD what we want (bypass, Qd, etc...).

There should be function(s) in DDInterface to tell DD what we want (bypass, Qd, etc...).