td-firmware / firmware

App

Clone Tools

Stats

Commits this week: 19
Total Committers: 15
Last Commit: 13 May
Commits this week: 19

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
Wednesday 13 May
11:51 am

Vendor - TEL - Vijay Pamula

LDT-3970: Updated Edit Treatmement parameter and individual parameter edit handling and addressed code review comments

Options
    • -1
    • +5
    ./Services/Messaging.c
    • -113
    • +392
    ./Services/TxParams.c
    • -34
    • +91
    ./Services/TxParams.h
10:48 am

Vendor - TEL - Varshini Nagabooshanam

Removedd

Removedd

10:44 am

Vendor - TEL - Varshini Nagabooshanam

added

added

10:43 am

Vendor - TEL - Varshini Nagabooshanam

LDT-2240-disposable-tube-set-self-test

Options
    • -14
    • +8
    ./Services/DrySelfTests.c
    • -1
    • +1
    ./Services/DrySelfTests.h
    • -0
    • +1
    ./Services/Messaging.c
Tuesday 12 May
10:38 pm

Vendor - TEL - Praneeth Bunne

LDT-1886: Refactor Auto-Eject back-off to retract ejector before reversing pump

Options
    • -31
    • +44
    ./Services/StateServices/TubeSetAutoEject.c
    • -1
    • +1
    ./Services/StateServices/TubeSetAutoEject.h
8:41 pm

Sean Nash and Vendor - TEL - Arpita Srivastava

8:40 pm

Vendor - TEL - Praneeth Bunne

LDT-1886: Add back-off state in auto-eject

Options
    • -9
    • +64
    ./Services/StateServices/TubeSetAutoEject.c
    • -0
    • +1
    ./Services/StateServices/TubeSetAutoEject.h
    • -15
    • +14
    ./Services/StateServices/TubeSetInstall.c
5:06 pm

Vendor - TEL - Varshini Nagabooshanam

LDT-2240: Review comments addressed

Options
    • -1
    • +1
    ./Services/DrySelfTests.c
    • -1
    • +1
    ./Services/DrySelfTests.h
3:47 pm

Vendor - TEL - Arpita Srivastava

Add @param section for reasons.

Add @param section for reasons.

3:44 pm

Vendor - TEL - Arpita Srivastava

Please list names of variables in inputs and outputs. You can move any description in the brief section.

Please list names of variables in inputs and outputs.
You can move any description in the brief section.

3:38 pm

Vendor - TEL - Arpita Srivastava

LDT-3505: Pulled latest staging into LDT-3505

Options
    • -0
    • +9
    ./Controllers/SyringePump.c
    • -11
    • +14
    ./Modes/ModeTreatment.c
    • -5
    • +6
    ./Services/AlarmMgmtSWFaults.h
    • -1
    • +2
    ./Services/Messaging.c
3:04 pm

Vendor - TEL - Varshini Nagabooshanam

LDT-2240-disposable-tube-set-self-test

Options
    • -6
    • +52
    ./Modes/ModePreTreat.c
    • -2
    • +4
    ./Modes/ModePreTreat.h
    • -2
    • +3
    ./Services/AlarmMgmtSWFaults.h
    • -77
    • +70
    ./Services/DrySelfTests.c
    • -1
    • +2
    ./Services/Messaging.c
2:24 pm

Vendor - TEL - Arpita Srivastava

Please add a comment for this check and break the line in two lines.

Please add a comment for this check and break the line in two lines.

2:24 pm

Vendor - TEL - Arpita Srivastava

Please add a comment about this condition check. Move this part : ( fabs( request.ufRate - validatedUFRateLhr ) < NEARLY_ZERO ) ) to the next line.

Please add a comment about this condition check.

Move this part : ( fabs( request.ufRate - validatedUFRateLhr ) < NEARLY_ZERO ) ) to the next line.

2:18 pm

Vendor - TEL - Arpita Srivastava

Can we please align (move more towards right) the comments of this newly added section with the comments of existing sections?

Can we please align (move more towards right) the comments of this newly added section with the comments of existing sections?

2:00 pm

Vendor - TEL - Arpita Srivastava

Please keep on declaration of the variables in this section and move the initialization to initTreatmentParameters

Please keep on declaration of the variables in this section and move the initialization to initTreatmentParameters

1:59 pm

Vendor - TEL - Arpita Srivastava

Please rename this function to initTreatmentParameters

Please rename this function to initTreatmentParameters

1:58 pm

Vendor - TEL - Arpita Srivastava

We only add File level or global variables in Inputs and Outputs. Please remove functions from the input section of this and other functions.

We only add File level or global variables in Inputs and Outputs.
Please remove functions from the input section of this and other functions.

1:56 pm

Vendor - TEL - Arpita Srivastava

Even if you do not have an Output, please add the tag with None as the value. Please fix in other functions as well.

Even if you do not have an Output, please add the tag with None as the value.
Please fix in other functions as well.

1:54 pm

Vendor - TEL - Arpita Srivastava

Please indent the doxygen comments of all the functions, similar to existing functions.

Please indent the doxygen comments of all the functions, similar to existing functions.

9:40 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

Monday 11 May
5:41 pm

Vendor - TEL - Arpita Srivastava

origTreatmentParams should be an output?

origTreatmentParams should be an output?

5:39 pm

Vendor - TEL - Arpita Srivastava

Please enclose each condition in parenthesis

Please enclose each condition in parenthesis

5:34 pm

Vendor - TEL - Arpita Srivastava

Can you please move the TODO comment above this line?

Can you please move the TODO comment above this line?

5:32 pm

Vendor - TEL - Arpita Srivastava

Please add a comment for this check.

Please add a comment for this check.

5:31 pm

Vendor - TEL - Arpita Srivastava

Please add @param

Please add @param

5:29 pm

Vendor - TEL - Arpita Srivastava

Please add @param

Please add @param

5:28 pm

Vendor - TEL - Arpita Srivastava

treatParamsConfirmed should be an output?

treatParamsConfirmed should be an output?

5:25 pm

Vendor - TEL - Arpita Srivastava

Delete extra line

Delete extra line

4:58 pm

Vendor - TEL - Arpita Srivastava

Please assign it to FALSE

Please assign it to FALSE