td-firmware / firmware

App

Clone Tools

Stats

Commits this week: 42
Total Committers: 18
Last Commit: 10:10
Commits this week: 42

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 14 Jul
10:36 am

Dara Navaei

Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

Options
    • -1
    • +1
    ./TDCommon.h
10:18 am

Vendor - TEL - Vijay Pamula

LDT-4028: Addressed Review Commentsfor Install state

Options
    • -1
    • +1
    ./Modes/ModePreTreat.c
    • -7
    • +7
    ./Services/StateServices/TubeSetInstall.c
10:05 am

Vendor - TEL - Varshini Nagabooshanam

LDT-4149: Driver Add ons

Options
    • -1
    • +1
    ./Drivers/PinchValve.c
10:02 am

Vendor - TEL - Varshini Nagabooshanam

LDT-4149: Driver Add ons

Options
    • -0
    • +3
    ./Tasks/TaskGeneral.c
    • -0
    • +3
    ./Tasks/TaskPriority.c
9:59 am

Vendor - TEL - Varshini Nagabooshanam

LDT-4149: Driver Add ons

Options
    • -193
    • +573
    ./Drivers/PinchValve.c
    • -40
    • +6
    ./Drivers/PinchValve.h
9:43 am

Vendor - TEL - Vijay Pamula

Merge branch 'staging' into LDT-4028-pre-treatment---fw-implementation

Options
    • -2
    • +3
    ./Services/Messaging.c
9:39 am

Sean Nash

Update all function headers in this file to Leahi style (e.g. \b before inputs/outputs).

Update all function headers in this file to Leahi style (e.g. \b before inputs/outputs).

9:38 am

Sean Nash

Add extra line (should be 2 lines) after test support banner.

Add extra line (should be 2 lines) after test support banner.

9:36 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:35 am

Sean Nash

Update function header format from Denali to Leahi style (e.g. \b for inputs/outputs) throughout this file.

Update function header format from Denali to Leahi style (e.g. \b for inputs/outputs) throughout this file.

9:26 am

Vendor - TEL - Arpita Srivastava

currentInstallState is an input.

currentInstallState is an input.

9:23 am

Vendor - TEL - Arpita Srivastava

Do we need doxygen comments for static function declarations? We will have a header in the definition.

Do we need doxygen comments for static function declarations?
We will have a header in the definition.

9:18 am

Vendor - TEL - Arpita Srivastava

AirPump.h should be above AirTrap.h

AirPump.h should be above AirTrap.h

9:17 am

Sean Nash

Monday 13 Jul
1:47 pm

Sean Nash

Who is calling this function? Should be coming from UI, right? So should be a message handler (returns BOOL, takes a message as param, function is in table at top of Messaging.c).

Who is calling this function? Should be coming from UI, right? So should be a message handler (returns BOOL, takes a message as param, function is in table at top of Messaging.c).

1:22 pm

Sean Nash

Need to set state to preload state I think. Otherwise you will go to interrupted state.

Need to set state to preload state I think. Otherwise you will go to interrupted state.

12:44 pm

Sean Nash

Why did you remote call to init function here? Is it initialized somewhere else?

Why did you remote call to init function here? Is it initialized somewhere else?

12:41 pm

Sean Nash

Use ternary operation to assign TRUE or FALSE explicitly.

Use ternary operation to assign TRUE or FALSE explicitly.

12:39 pm

Sean Nash

This is at end of enum list above. So it should be at end of table here.

This is at end of enum list above. So it should be at end of table here.

9:36 am

Vendor - TEL - Vijay Pamula

LDT-4029: Addressed Review comments for Heparin Setup State

Options
    • -7
    • +17
    ./Modes/ModePreTreat.c
    • -124
    • +69
    ./Modes/StatePreTxHeparinSetup.c
    • -12
    • +0
    ./Modes/StatePreTxHeparinSetup.h
Friday 10 Jul
3:33 pm

Vendor - TEL - Praneeth Bunne

it will be adjusted when I merge latest staging

it will be adjusted when I merge latest staging

3:31 pm

Vendor - TEL - Praneeth Bunne

not required anymore, forgot to remove

not required anymore, forgot to remove

3:21 pm

Sean Nash

I think a comment here is needed. Keep it and maybe mention rinseback alarm responses too.

I think a comment here is needed. Keep it and maybe mention rinseback alarm responses too.

3:20 pm

Sean Nash

Maybe this should be handled in OperationModes exec since it applies to all confirmations (not just alarm related ones).

Maybe this should be handled in OperationModes exec since it applies to all confirmations (not just alarm related ones).

3:19 pm

Sean Nash

There will be confirmations that are not handled here (e.g. power off). So we shouldn't fault in default - we should do nothing.

There will be confirmations that are not handled here (e.g. power off). So we shouldn't fault in default - we should do nothing.

3:17 pm

Sean Nash

Shutdown would be handled somewhere else (buttons.c probably). It's not related to alarms.

Shutdown would be handled somewhere else (buttons.c probably). It's not related to alarms.

3:09 pm

Sean Nash

Is this still possible since we're not returning to TxPaused from rinseback anymore?

Is this still possible since we're not returning to TxPaused from rinseback anymore?

2:31 pm

Vendor - TEL - Varshini Nagabooshanam

New Driver Add ons

Options
    • -2
    • +2
    ./Drivers/PinchValve.h
2:01 pm

Vendor - TEL - Varshini Nagabooshanam

New Driver Add ons

Options
    • -420
    • +600
    ./Drivers/PinchValve.c
    • -30
    • +49
    ./Drivers/PinchValve.h
  3. … 1 more file in changeset.
11:17 am

Sean Nash

Is rinseback completed an indication that we've done a blood prime following a temporary break? If so, I guess this makes sense, but we have to do more than increase the UF volume. We have to adjus...

Is rinseback completed an indication that we've done a blood prime following a temporary break?
If so, I guess this makes sense, but we have to do more than increase the UF volume. We have to adjust the UF rate so that we will reach the new (larger) UF volume when the treatment completes.
We also need to add RB volume to UF volume in pre-tx when treatment parameters are given/validated (for the final rinseback which we will always have so we should just plan for it before treatment starts).