leahi-common

Clone Tools

Stats

Commits this week: 10
Total Committers: 11
Last Commit: 11 Dec
Commits this week: 10

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
Thursday 11 Dec
5:23 pm

Varshini Nagabooshanam <ven.vnagabooshanam@diality.com>

Merged staging

Options
    • -3
    • +6
    /MsgDefs.h
2:00 pm

Sean Nash

Is this temporary? If so, put a TODO to remove later.

Is this temporary? If so, put a TODO to remove later.

1:59 pm

Sean Nash

Remove extra spaces between { and msg name.

Remove extra spaces between { and msg name.

1:58 pm

Sean Nash

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC e...

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC exec).

1:56 pm

Sean Nash

Do these belong in the spent chamber fill group?

Do these belong in the spent chamber fill group?

10:15 am

Vendor - TEL - Sameer Poyil

move static function after public function

move static function after public function

10:14 am

Vendor - TEL - Sameer Poyil

let public function on top

let public function on top

10:11 am

Vendor - TEL - Sameer Poyil

Let all public functions come on top for all files

Let all public functions come on top for all files

Tuesday 09 Dec
11:25 am

Raghu Kallala

11:24 am

Dara Navaei

11:20 am

Sean Nash and Vendor - TEL - Sameer Poyil

11:19 am

Dara Navaei

11:15 am

Dara Navaei

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

Options
    • -2
    • +2
    /AlarmDefs.h
    • -2
    • +2
    /MsgDefs.h
11:15 am

Dara Navaei

Done

Done

11:12 am

Vendor - TEL - Sameer Poyil

comments should start with capital letter

comments should start with capital letter

10:59 am

Raghu Kallala

LDT-2936 merge into staging

Options
    • -9
    • +12
    /AlarmDefs.h
    • -0
    • +3
    /MsgDefs.h
10:43 am

Sean Nash , Vinayakam Mani and Michael Garthwaite

10:29 am

Raghu Kallala

Resolved

Resolved

10:27 am

Sean Nash

Shouldn't EVERY state set ALL valves?

Shouldn't EVERY state set ALL valves?

10:16 am

Sean Nash

Change comment from 3 to 5 minutes to match code.

Change comment from 3 to 5 minutes to match code.

10:14 am

Sean Nash

I don't see fix

I don't see fix

10:03 am

Raghu Kallala

Fixed the ordering

Fixed the ordering

9:58 am

Sean Nash

I didn't see this in header file.

I didn't see this in header file.

9:57 am

Sean Nash

I don't see these Lower/Upper Range Limit functions declared in the header file.

I don't see these Lower/Upper Range Limit functions declared in the header file.

9:56 am

Raghu Kallala

LDT-2936 fixed alarm order and removed waitForTankFill from concentrate flush states

Options
    • -1
    • +1
    /AlarmDefs.h
    • -1
    • +0
    /FPDefs.h
9:33 am

Sean Nash

Need to delete this unit (ModeTxParams.c/.h). Also need to remove the mode from the modes enumeration and from OperationModes.c.

Need to delete this unit (ModeTxParams.c/.h). Also need to remove the mode from the modes enumeration and from OperationModes.c.

9:32 am

Sean Nash

I don't think we need this signal function since this is not a mode anymore. Is this function being called somewhere?

I don't think we need this signal function since this is not a mode anymore. Is this function being called somewhere?

Monday 08 Dec
7:02 pm

Vendor - TEL - Sameer Poyil

Suggestive comment => can we make it as a run time configuration option so that we don't need to change the code ?

Suggestive comment => can we make it as a run time configuration option so that we don't need to change the code ?

7:00 pm

Vendor - TEL - Sameer Poyil

Let public function on top

Let public function on top

6:53 pm

Vendor - TEL - Sameer Poyil

Suggestive comment => Shall we do it as a run time configurable so that we don't need to change the code?

Suggestive comment => Shall we do it as a run time configurable so that we don't need to change the code?