td-firmware / firmware / App

Monitors

Clone Tools

Stats

Commits this week: 0
Total Committers: 9
Last Commit: 10:36
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
Today
1 hr ago

Sean Nash

2 hrs ago

Vendor - TEL - Arpita Srivastava

currentInstallState is an input.

currentInstallState is an input.

2 hrs ago

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.

2 hrs ago

Vendor - TEL - Arpita Srivastava

AirPump.h should be above AirTrap.h

AirPump.h should be above AirTrap.h

Tuesday 07 Jul
3:16 pm

Sean Nash

Thursday 02 Jul
11:24 am

Sean Nash

Wednesday 01 Jul
10:21 am

Sean Nash

Remove extra ;

Remove extra ;

10:18 am

Sean Nash

Can we align comments?

Can we align comments?

10:01 am

Sean Nash

Missing pinch valves (H1 and H19).

Missing pinch valves (H1 and H19).

9:59 am

Sean Nash

Remove this blank line.

Remove this blank line.

Saturday 27 Jun
9:11 am

Sean Nash

Where is testBatteryI2cStatusOverride (from BatteryDriver)?

Where is testBatteryI2cStatusOverride (from BatteryDriver)?

Friday 26 Jun
3:16 pm

Sean Nash

Add blank line before return statement.

Add blank line before return statement.

9:51 am

Vendor - TEL - Vijay Pamula

LDT-4028:Addressed code review commens for Pre-Tx Install State

Options
    • -2
    • +2
    ./Pressures.c
  2. … 8 more files in changeset.
9:33 am

suresh

LDT-4072: added publish interval override

Options
    • -18
    • +55
    ./Battery.c
    • -0
    • +1
    ./Battery.h
  3. … 1 more file in changeset.
Wednesday 24 Jun
2:42 pm

Vendor - TEL - Arpita Srivastava

please add message id.

please add message id.

2:34 pm

Sean Nash

We should set all TD actuators here, not just blood pump.

We should set all TD actuators here, not just blood pump.

2:28 pm

Vendor - TEL - Arpita Srivastava

please remove extra line

please remove extra line

2:26 pm

Vendor - TEL - Arpita Srivastava

Output is none

Output is none

2:23 pm

Vendor - TEL - Arpita Srivastava

Please add message id.

Please add message id.

2:22 pm

Vendor - TEL - Arpita Srivastava

Output is none

Output is none

2:21 pm

Vendor - TEL - Arpita Srivastava

Message id MSG_ID_OFF_BUTTON_PRESS_REQUEST looks wrong

Message id MSG_ID_OFF_BUTTON_PRESS_REQUEST looks wrong

1:14 pm

Sean Nash

I think we should accept this as long as we're in pre-tx: install state. User might confirm before auto-load finishes and that's ok.

I think we should accept this as long as we're in pre-tx: install state. User might confirm before auto-load finishes and that's ok.

1:00 pm

Sean Nash

Change "SetupConnection" to "SetupTubingSetConnection"

Change "SetupConnection" to "SetupTubingSetConnection"

12:59 pm

Sean Nash

Recommend changing "SETUP_CONNECTION" to "SETUP_TUBING_SET_CONNECTIONS" in these 2 msg ID names.

Recommend changing "SETUP_CONNECTION" to "SETUP_TUBING_SET_CONNECTIONS" in these 2 msg ID names.

9:53 am

suresh

Updated battery monitor per review feedback

Options
    • -154
    • +164
    ./Battery.c
    • -15
    • +13
    ./Battery.h
  3. … 3 more files in changeset.
Monday 22 Jun
5:39 pm

Sean Nash

Can we restore this code?

Can we restore this code?

5:36 pm

Sean Nash

Remove FANS test.

Remove FANS test.

5:34 pm

Sean Nash

Remove.

Remove.

5:33 pm

Sean Nash

Move this to Messaging.c

Move this to Messaging.c

5:32 pm

Sean Nash

Remove. Replaced by generic confirmation functions and messages.

Remove. Replaced by generic confirmation functions and messages.