td-firmware

Clone Tools

Stats

Commits this week: 14
Total Committers: 13
Last Commit: 11:56
Commits this week: 14

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
6 hrs ago

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/TDCommon.h
6 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Merge branch 'LDT-1840-Transmembrane-Pressure-update' into develop

Options
    • -0
    • +1
    /firmware/App/Monitors/Pressures.c
6 hrs ago

Zoltan Miskolci , Vendor - TEL - Jashwant Gantyada and Sean Nash

6 hrs ago

Vendor - TEL - Varshini Nagabooshanam

LDT-1840-Transmembrane-Pressure-update

Options
    • -0
    • +1
    /firmware/App/Monitors/Pressures.c
    • -0
    • +1
    /firmware/App/Monitors/Pressures.h
9 hrs ago

Raghu Kallala

LDT-473 fixed typo and addressed review comments

Options
    • -4
    • +4
    /firmware/App/Monitors/Pressures.c
    • -2
    • +2
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -3
    • +1
    /firmware/App/Services/TxParams.c
11 hrs ago

Raghu Kallala

Had that question. Thanks Changed to else block

Had that question. Thanks
Changed to else block

11 hrs ago

Raghu Kallala

Updated

Updated

11 hrs ago

Raghu Kallala

This will be gone once we have institutional records, do you still want me to move it to a separate func?

This will be gone once we have institutional records, do you still want me to move it to a separate func?

11 hrs ago

Raghu Kallala

Fixed

Fixed

Wednesday 18 Feb
10:09 pm

Sean Nash

Can we uncomment these and send msg?

Can we uncomment these and send msg?

10:08 pm

Sean Nash

Can we uncomment this and send the limits to the UI?

Can we uncomment this and send the limits to the UI?

10:06 pm

Sean Nash

Should we cast these to (S32)?

Should we cast these to (S32)?

10:04 pm

Sean Nash

These 2 last values have already been used above. Change to 154 and 155.

These 2 last values have already been used above. Change to 154 and 155.

10:02 pm

Sean Nash

Can this stuff down here be done in a separate function that is called here? This function is too big.

Can this stuff down here be done in a separate function that is called here? This function is too big.

10:02 pm

Sean Nash

Should this just be an "else" to above condition?

Should this just be an "else" to above condition?

9:58 pm

Sean Nash

When do you plan on removing this?

When do you plan on removing this?

9:56 pm

Sean Nash

Just a heads up that all message handling functions in Leahi have the same prototype - just different function names (i.e. BOOL funcName( MESSAGE_T *message)) and that includes all Dialin command m...

Just a heads up that all message handling functions in Leahi have the same prototype - just different function names (i.e. BOOL funcName( MESSAGE_T *message)) and that includes all Dialin command message handlers like these.

9:48 pm

Sean Nash

This param has been renamed to HEPARIN_STOP_TIME which is a couple rows below so we don't need to add this.

This param has been renamed to HEPARIN_STOP_TIME which is a couple rows below so we don't need to add this.

9:47 pm

Sean Nash

We will not have this saline bolus state in Leahi.

We will not have this saline bolus state in Leahi.

9:45 pm

Sean Nash

This alarm does not belong here. Put it back where it was before. In the next table below, the alarms are sorted by rank. But this property table is sorted by alarm ID #.

This alarm does not belong here. Put it back where it was before.
In the next table below, the alarms are sorted by rank. But this property table is sorted by alarm ID #.

9:43 pm

Sean Nash

Why did you move it. The alarm is #30 so it should stay here where you had it before.

Why did you move it. The alarm is #30 so it should stay here where you had it before.

1:47 pm

Raghu Kallala

LDT-473 cleaned S32 related functions and updated tx param payload

Options
    • -3
    • +3
    /firmware/App/Modes/ModeTreatment.c
    • -228
    • +77
    /firmware/App/Services/TxParams.c
    • -17
    • +24
    /firmware/App/Services/TxParams.h
11:32 am

Vendor - TEL - Arpita Srivastava

This code review contains known issues, and it is not yet ready for a detailed review.

This code review contains known issues, and it is not yet ready for a detailed review.

11:29 am

Vendor - TEL - Arpita Srivastava

LDT-3505: Added Syringe Pump

Options
    • -0
    • +2829
    /firmware/App/Controllers/SyringePump.c
    • -0
    • +178
    /firmware/App/Controllers/SyringePump.h
    • -0
    • +4
    /firmware/App/Services/AlarmMgmtSWFaults.h
Tuesday 17 Feb
2:13 pm

Raghu Kallala

WIP restructured the not applicable param validation

Options
    • -0
    • +1
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -78
    • +113
    /firmware/App/Services/TxParams.c
1:32 pm

Raghu Kallala

fixed

fixed

1:32 pm

Raghu Kallala

reworded

reworded

1:32 pm

Raghu Kallala

Updated

Updated

11:48 am

Vendor - TEL - Varshini Nagabooshanam

Based on the Denali properties, I have written the following.

Based on the Denali properties, I have written the following.

11:06 am

Raghu Kallala

replace with else block

replace with else block