td-firmware / firmware / App

Modes

Clone Tools

Stats

Commits this week: 6
Total Committers: 13
Last Commit: 09 Jul
Commits this week: 6

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 09 Jul
2:07 pm

Sean Nash

What alarm is this? It doesn't look like you've added a new alarm. This is one of the reserved (unused) alarms that UI s/w might use.

What alarm is this? It doesn't look like you've added a new alarm. This is one of the reserved (unused) alarms that UI s/w might use.

1:36 pm

Sean Nash

Do we really need these setup functions for each state? They all look very simple - just 1 or 2 lines of code.

Do we really need these setup functions for each state? They all look very simple - just 1 or 2 lines of code.

1:33 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-4040_Rinseback - FW Implementation - 3/10:
LEAHI-TD-FIRMWARE-LDT-4040_Rinseback - FW Implementation - 3/10:
1:32 pm

Sean Nash

I believe the SyringePump controller will trigger this alarm at end of seek operation.

I believe the SyringePump controller will trigger this alarm at end of seek operation.

1:28 pm

Sean Nash

Syringe pump should already be stopped if preload is done. Do we need to call this here?

Syringe pump should already be stopped if preload is done. Do we need to call this here?

1:24 pm

Sean Nash

This is called from preload state. Not needed here I think.

This is called from preload state. Not needed here I think.

1:22 pm

Sean Nash

Don't put comments on function prototypes. Doxygen gets details of functions from the function header.

Don't put comments on function prototypes. Doxygen gets details of functions from the function header.

1:20 pm

Sean Nash

Why (void)?

Why (void)?

1:16 pm

Sean Nash

Where is this being called? Should be called from pre-treatment init I think.

Where is this being called? Should be called from pre-treatment init I think.

1:13 pm

Sean Nash

Confirmation signal sounds like a message handler from UI. Message handlers must accept a message as parameter and return BOOL. They must also be added to the message handler table in Messaging.c.

Confirmation signal sounds like a message handler from UI. Message handlers must accept a message as parameter and return BOOL. They must also be added to the message handler table in Messaging.c.

1:11 pm

Sean Nash

Why are we signaling syringe load confirmed here?

Why are we signaling syringe load confirmed here?

1:11 pm

Sean Nash

Do these states need to be public?

Do these states need to be public?

1:08 pm

Sean Nash

Move up one row to alphabetize.

Move up one row to alphabetize.

11:44 am

Vendor - TEL - Praneeth Bunne

LDT-4040: Add rinseback realted code

Options
    • -77
    • +167
    ./ModeTreatment.c
    • -0
    • +5
    ./ModeTreatment.h
    • -82
    • +129
    ./OperationModes.c
    • -0
    • +15
    ./OperationModes.h
    • -0
    • +1
    ./StateTxBloodPrime.c
    • -0
    • +1
    ./StateTxDialysis.c
    • -1
    • +3
    ./StateTxPaused.c
    • -0
    • +584
    ./StateTxRecirc.c
    • -0
    • +51
    ./StateTxRecirc.h
    • -0
    • +969
    ./StateTxRinseback.c
    • -0
    • +67
    ./StateTxRinseback.h
  12. … 6 more files in changeset.
9:40 am

Vendor - TEL - Vijay Pamula

LDT-4029: Pre-Treatment Heparin Setup State Implementation

Options
    • -5
    • +55
    ./ModePreTreat.c
    • -1
    • +2
    ./ModePreTreat.h
    • -0
    • +539
    ./StatePreTxHeparinSetup.c
    • -0
    • +62
    ./StatePreTxHeparinSetup.h
  5. … 1 more file in changeset.
Tuesday 07 Jul
3:16 pm

Sean Nash

3:16 pm

Sean Nash

3:10 pm

Vendor - TEL - Vijay Pamula

LDT-4028: Adressed Review Comment foro Install State

Options
    • -1
    • +0
    ./ModePreTreat.c
10:16 am

Vendor - TEL - Vijay Pamula

LDT-4028: Merge Latest Staging

Options
    • -24
    • +98
    ./ModeStandby.c
  2. … 4 more files in changeset.
9:38 am

Vendor - TEL - Vijay Pamula

LDT-4028:Addresses Review Comments for Install State

Options
    • -4
    • +11
    ./ModePreTreat.c
  2. … 2 more files in changeset.
Monday 06 Jul
1:07 pm

Sean Nash

I would keep blank line between library includes and regular includes.

I would keep blank line between library includes and regular includes.

1:05 pm

Sean Nash

We can't have 2 alarms with ID 196. The enum only has the empty saline bag alarm. Looks like alarm 197 is duplicate of this 196 alarm you've added. I think you should just remove this new alarm 196...

We can't have 2 alarms with ID 196. The enum only has the empty saline bag alarm. Looks like alarm 197 is duplicate of this 196 alarm you've added.
I think you should just remove this new alarm 196 that you've added here.

10:56 am

Vendor - TEL - Praneeth Bunne

Friday 03 Jul
1:33 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./ModeStandby.c
    • -2
    • +2
    ./ModeStandby.h
    • -1
    • +1
    ./StateTxPaused.c
  4. … 12 more files in changeset.
Thursday 02 Jul
3:06 pm

Vendor - TEL - Jashwant Gantyada and Sean Nash

2:59 pm

Vendor - TEL - Praneeth Bunne

LDT-4013-1: Address CR comments

Options
    • -5
    • +5
    ./ModeStandby.c
    • -1
    • +1
    ./ModeStandby.h
2:42 pm

Sean Nash

Align =

Align =

2:41 pm

Sean Nash

Should modality be an enum type?

Should modality be an enum type?

2:41 pm

Sean Nash

Align =

Align =

2:40 pm

Sean Nash

Should this be an enum type?

Should this be an enum type?