td-firmware / firmware / App / Modes

StatePreTxHeparinSetup.c

Clone Tools

Stats

Commits this week: 2
Total Committers: 2
Last Commit: 18 Aug
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
Thursday 13 Aug
1:00 pm

Vendor - TEL - Vijay Pamula

11:41 am

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -2
    • +2
    ./StatePreTxHeparinSetup.c
  2. … 9 more files in changeset.
11:15 am

Sean Nash and Vendor - TEL - Arpita Srivastava

10:16 am

Vendor - TEL - Vijay Pamula

LDT-4029:Addressed review comments

Options
    • -2
    • +4
    ./StatePreTxHeparinSetup.c
9:53 am

Vendor - TEL - Arpita Srivastava

Add to inputs : heparinSetupResumeRequested

Add to inputs : heparinSetupResumeRequested

9:51 am

Vendor - TEL - Arpita Srivastava

Add @alarm

Add @alarm

9:47 am

Vendor - TEL - Arpita Srivastava

Add to inputs : interruptedPreTxHeparinSetupState

Add to inputs : interruptedPreTxHeparinSetupState

Wednesday 12 Aug
4:33 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Merge latest staging

Options
    • -1
    • +1
    ./StatePreTxHeparinSetup.c
  2. … 6 more files in changeset.
2:31 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Updated heparin setup state

Options
    • -2
    • +2
    ./StatePreTxHeparinSetup.c
2:25 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Updated Heparin setup state

Options
    • -5
    • +13
    ./StatePreTxHeparinSetup.c
  2. … 1 more file in changeset.
10:26 am

Sean Nash

Does default mean invalid state (s/w fault) or does it mean it's a state that we don't need to do anything for? If latter, add a comment here saying that.

Does default mean invalid state (s/w fault) or does it mean it's a state that we don't need to do anything for? If latter, add a comment here saying that.

10:25 am

Sean Nash

Use if else or ternary operator to set BOOL to TRUE or FALSE explicitly.

Use if else or ternary operator to set BOOL to TRUE or FALSE explicitly.

10:21 am

Sean Nash

Add a TODO comment to adjust this when new syringe pump assembly is ready.

Add a TODO comment to adjust this when new syringe pump assembly is ready.

9:25 am

Vendor - TEL - Vijay Pamula

LDT-4029:Updated Heparin setup state

Options
    • -26
    • +12
    ./StatePreTxHeparinSetup.c
  2. … 3 more files in changeset.
Wednesday 05 Aug
12:45 pm

Sean Nash

What if we're in paused state (because your alarm has stop property set)? Should we return TRUE here if in paused state and interrupted state was the await syringe load confirmation state?

What if we're in paused state (because your alarm has stop property set)? Should we return TRUE here if in paused state and interrupted state was the await syringe load confirmation state?

9:51 am

Vendor - TEL - Vijay Pamula

LDT-4029:Updated heparin setup state

Options
    • -4
    • +5
    ./StatePreTxHeparinSetup.c
Friday 31 Jul
7:06 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Addressed review comments

Options
    • -3
    • +3
    ./StatePreTxHeparinSetup.c
4:24 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Addressed review comments

Options
    • -2
    • +3
    ./StatePreTxHeparinSetup.c
Thursday 30 Jul
3:37 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Updated heparin setup state

Options
    • -7
    • +10
    ./StatePreTxHeparinSetup.c
10:31 am

Sean Nash

Set "Stops" property back to TRUE like the other reserved UI alarms.

Set "Stops" property back to TRUE like the other reserved UI alarms.

Wednesday 29 Jul
10:42 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Addressed review comments

Options
    • -6
    • +30
    ./StatePreTxHeparinSetup.c
  2. … 1 more file in changeset.
Tuesday 28 Jul
2:47 pm

Vendor - TEL - Arpita Srivastava

break this line

break this line

2:39 pm

Vendor - TEL - Arpita Srivastava

add an empty else block after the last else if block in these functions handlePreTxHeparinPreloadState handlePreTxHeparinAwaitSyringeLoadConfirmationState handlePreTxHeparinSeekState handlePreTxHep...

add an empty else block after the last else if block in these functions
handlePreTxHeparinPreloadState
handlePreTxHeparinAwaitSyringeLoadConfirmationState
handlePreTxHeparinSeekState
handlePreTxHeparinPrimeState
handlePreTxHeparinBolusState

2:32 pm

Vendor - TEL - Arpita Srivastava

break this line

break this line

10:44 am

Vendor - TEL - Arpita Srivastava

Add initialization for heparinSetupResumeRequested and syringeLoadConfirmed

Add initialization for heparinSetupResumeRequested and syringeLoadConfirmed

10:42 am

Vendor - TEL - Arpita Srivastava

Align all the function comments

Align all the function comments

10:19 am

Vendor - TEL - Arpita Srivastava

Delete this row.

Delete this row.

Monday 27 Jul
12:54 pm

Vendor - TEL - Vijay Pamula

LDT-4029:Updated heparin setup state

Options
    • -2
    • +2
    ./StatePreTxHeparinSetup.c
Sunday 26 Jul
8:37 pm

Sean Nash

Should this be the "Load Syringe" alarm that you just added?

Should this be the "Load Syringe" alarm that you just added?

Tuesday 21 Jul
5:45 pm

Sean Nash

I think ( setTxDur - preStop ) should be changed to ( preStop ). Also, I think preStop should be renamed to hepDurMin.

I think ( setTxDur - preStop ) should be changed to ( preStop ).
Also, I think preStop should be renamed to hepDurMin.