leahi-common

Clone Tools

Stats

Commits this week: 15
Total Committers: 20
Last Commit: 05 Jun
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
Today
1 min ago

Vendor - TEL - Arpita Srivastava

Add fluidType to inputs

Add fluidType to inputs

1 min ago

Vendor - TEL - Arpita Srivastava

Outputs: Add targetRinsebackVolumePlusAdditional_mL Update spelling for :cummulativeRinsebackVolume_mL

Outputs:
Add targetRinsebackVolumePlusAdditional_mL
Update spelling for :cummulativeRinsebackVolume_mL

3 mins ago

Dara Navaei

If a retry goes wrong will you trigger an alarm?

If a retry goes wrong will you trigger an alarm?

7 mins ago

Vendor - TEL - Arpita Srivastava

break this line

break this line

8 mins ago

Vendor - TEL - Arpita Srivastava

Add @Alarms

Add @Alarms

9 mins ago

Vendor - TEL - Arpita Srivastava

Add @Alarms

Add @Alarms

11 mins ago

Dara Navaei

Any fault alarm here?

Any fault alarm here?

11 mins ago

Vendor - TEL - Arpita Srivastava

fluidType is an input

fluidType is an input

11 mins ago

Vendor - TEL - Arpita Srivastava

break this line

break this line

12 mins ago

Vendor - TEL - Arpita Srivastava

Generally we put all the static functions after all the public functions. Can we reorganize the file as : Public functions Static functions Public Test functions

Generally we put all the static functions after all the public functions.
Can we reorganize the file as :
Public functions
Static functions
Public Test functions

15 mins ago

Vendor - TEL - Arpita Srivastava

Initialization is missing in the init function for variables from line 72 to 77.

Initialization is missing in the init function for variables from line 72 to 77.

16 mins ago

Vendor - TEL - Arpita Srivastava

Initialize in init.

Initialize in init.

17 mins ago

Vendor - TEL - Arpita Srivastava

Initialize in init. Check SyringPump.c for reference.

Initialize in init. Check SyringPump.c for reference.

24 mins ago

Dara Navaei

Have these variables been initialized correctly?

Have these variables been initialized correctly?

52 mins ago

Vendor - TEL - Arpita Srivastava

please add a TODO or remove commented code.

please add a TODO or remove commented code.

52 mins ago

Vendor - TEL - Arpita Srivastava

please add a TODO or remove commented code.

please add a TODO or remove commented code.

53 mins ago

Vendor - TEL - Arpita Srivastava

add resumeTreatmentAlarmResponseRequest to output

add resumeTreatmentAlarmResponseRequest to output

55 mins ago

Vendor - TEL - Arpita Srivastava

Update input - output list.

Update input - output list.

1 hr ago

Vendor - TEL - Arpita Srivastava

Add presUFRateMlMin to outputs

Add presUFRateMlMin to outputs

1 hr ago

Vendor - TEL - Arpita Srivastava

recirculateToBloodPrimeRequest is output.

recirculateToBloodPrimeRequest is output.

1 hr ago

Vendor - TEL - Arpita Srivastava

Add initialization for rinsebackToRecircRequest and recirculateToBloodPrimeRequest

Add initialization for rinsebackToRecircRequest and recirculateToBloodPrimeRequest

1 hr ago

Sean Nash

So then no need to initialize it here too. Remove " = FALSE".

So then no need to initialize it here too. Remove " = FALSE".

1 hr ago

Sean Nash

1 hr ago

Sean Nash

Wednesday 22 Jul
9:28 pm

Vendor - TEL - Suresh Dharnala

I agree but it is already in the init function.

I agree but it is already in the init function.

1:26 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

1:13 pm

Sean Nash

We generally don't add { } to switch cases unless they require local variable declarations.

We generally don't add { } to switch cases unless they require local variable declarations.

1:08 pm

Sean Nash

If we're not including a timestamp, can we just remove this line of code?

If we're not including a timestamp, can we just remove this line of code?

1:07 pm

Sean Nash

Why commented out?

Why commented out?

11:32 am

Sean Nash

Swap order of == for these 2 if statements - put constant on left side.

Swap order of == for these 2 if statements - put constant on left side.