td-firmware / firmware

App

Clone Tools

Stats

Commits this week: 15
Total Committers: 8
Last Commit: 19 Dec
Commits this week: 15

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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 19 Dec
2:14 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./TDCommon.h
1:57 pm

Dara Navaei

1:51 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./Controllers/AirPump.c
    • -1
    • +1
    ./Controllers/AirTrap.c
    • -1
    • +1
    ./Controllers/Ejector.c
    • -1
    • +1
    ./Controllers/Ejector.h
    • -1
    • +1
    ./Controllers/Valves.c
    • -1
    • +1
    ./Controllers/Valves.h
    • -2
    • +2
    ./Drivers/EjectorMotor.c
    • -2
    • +2
    ./Drivers/EjectorMotor.h
    • -1
    • +1
    ./Drivers/PressureSensor.c
    • -1
    • +1
    ./Drivers/RotaryValve.c
    • -1
    • +1
    ./Drivers/RotaryValve.h
    • -1
    • +1
    ./Services/AlarmMgmtSWFaults.h
    • -1
    • +1
    ./Services/FpgaTD.c
    • -1
    • +1
    ./Services/FpgaTD.h
    • -1
    • +1
    ./Services/Messaging.c
    • -1
    • +1
    ./TDCommon.h
1:48 pm

Dara Navaei

Merge branch 'staging' into LDT-1886-blood-set-auto-load-and-auto-eject-implement-ejector-sensor

Options
    • -5
    • +5
    ./Controllers/AirPump.c
    • -5
    • +5
    ./Controllers/AirTrap.c
    • -4
    • +4
    ./Controllers/Ejector.c
    • -4
    • +4
    ./Controllers/Ejector.h
    • -5
    • +5
    ./Controllers/Valves.c
    • -5
    • +5
    ./Controllers/Valves.h
    • -4
    • +4
    ./Drivers/EjectorMotor.c
    • -4
    • +4
    ./Drivers/EjectorMotor.h
    • -4
    • +4
    ./Drivers/PressureSensor.c
    • -5
    • +5
    ./Drivers/RotaryValve.c
    • -5
    • +5
    ./Drivers/RotaryValve.h
    • -5
    • +5
    ./Services/AlarmMgmtSWFaults.h
    • -5
    • +5
    ./Services/FpgaTD.c
    • -5
    • +5
    ./Services/FpgaTD.h
    • -5
    • +5
    ./Services/Messaging.c
    • -9
    • +9
    ./TDCommon.h
12:16 pm

Vendor - TEL - Sameer Poyil and Sean Nash

9:44 am

Dara Navaei

LDT-3088 added a macro switch to convert the pressure sensors to 15psi range from 30psi once the hardware is updated

Options
    • -0
    • +7
    ./Drivers/PressureSensor.c
Thursday 18 Dec
3:29 pm

Dara Navaei

LDT-1886 added doxygen comments

Options
    • -3
    • +3
    ./Drivers/EjectorMotor.h
3:24 pm

Dara Navaei

LDT-1886 addressed the code review comment

Options
    • -1
    • +1
    ./TDCommon.h
3:08 pm

Dara Navaei

We tend to initialize the local variables as a standard practice.

We tend to initialize the local variables as a standard practice.

3:01 pm

Dara Navaei

LDT-1886 updated the valves code to be able to rehome the valves if needed

Options
    • -2
    • +4
    ./Controllers/Valves.c
    • -1
    • +1
    ./TDCommon.h
2:54 pm

Sean Nash

These don't need to be initialized to zero.

These don't need to be initialized to zero.

Wednesday 17 Dec
1:50 pm

Dara Navaei

LDT-1886 fixed the valves round down the encoder to division of 8 all the time

Options
    • -8
    • +2
    ./Controllers/Valves.c
11:11 am

Dara Navaei

LDT-1886 addressed the code review comments

Options
    • -31
    • +31
    ./Controllers/Ejector.c
    • -25
    • +25
    ./Controllers/Valves.c
    • -30
    • +30
    ./Drivers/RotaryValve.c
11:07 am

Dara Navaei

Done

Done

11:06 am

Dara Navaei

Done

Done

11:05 am

Dara Navaei

Done

Done

11:04 am

Dara Navaei

LDT-1886 addressed the code review comments

Options
    • -37
    • +22
    ./Controllers/Valves.c
10:49 am

Dara Navaei

Done

Done

10:48 am

Dara Navaei

Done

Done

Tuesday 16 Dec
2:55 pm

Dara Navaei

Done.

Done.

2:53 pm

Dara Navaei

No. It is permanent, I would like to know the value for the homing.

No. It is permanent, I would like to know the value for the homing.

2:17 pm

Sean Nash

The else in this trinary looks wrong - doesn't round up to multiple of 8.

The else in this trinary looks wrong - doesn't round up to multiple of 8.

2:15 pm

Sean Nash

This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).

This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).

2:13 pm

Sean Nash

Declarations should be at top of scope.

Declarations should be at top of scope.

2:13 pm

Sean Nash

Is this temporary?

Is this temporary?

2:12 pm

Sean Nash

2:08 pm

Vendor - TEL - Varshini Nagabooshanam

Updated.

Updated.

2:07 pm

Vendor - TEL - Varshini Nagabooshanam

Yes, we need to

Yes, we need to

2:07 pm

Vendor - TEL - Varshini Nagabooshanam

Deleted.

Deleted.

1:47 pm

Dara Navaei

LDT-1886 updated the valves homing sequence

Options
    • -70
    • +159
    ./Controllers/Valves.c
    • -0
    • +1
    ./Controllers/Valves.h
    • -1
    • +0
    ./Drivers/EjectorMotor.c
    • -0
    • +1
    ./Drivers/RotaryValve.c
    • -3
    • +2
    ./Drivers/RotaryValve.h
    • -0
    • +2
    ./Services/AlarmMgmtSWFaults.h