leahi-common

Clone Tools

Stats

Commits this week: 28
Total Committers: 19
Last Commit: 12 Jun
Commits this week: 28

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
Thursday 11 Jun
2:27 pm

Dara Navaei

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

Options
    • -2
    • +2
    /AlarmDefs.h
    • -2
    • +2
    /MsgDefs.h
2:24 pm

Michael Garthwaite

2:23 pm

Michael Garthwaite

Merge branch 'LDT-4001-hdf-fw-implementation-2-4' into staging

Options
1:27 pm

Vinayakam Mani and Sean Nash

11:20 am

Vendor - TEL - Arpita Srivastava

please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

11:20 am

Vendor - TEL - Arpita Srivastava

please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

11:13 am

Vendor - TEL - Arpita Srivastava

Alarm id should be 196? Description looks same as 194.

Alarm id should be 196?
Description looks same as 194.

10:34 am

Vendor - TEL - Varshini Nagabooshanam

Fixed

Options
    • -14
    • +18
    /AlarmDefs.h
10:08 am

Michael Garthwaite

fixed thanks

fixed thanks

10:07 am

Michael Garthwaite

LDT-4001: code review comments

Options
    • -1
    • +1
    /MsgDefs.h
10:00 am

Sean Nash

Should we change msg ID from speed to rate?

Should we change msg ID from speed to rate?

9:59 am

Michael Garthwaite

fixed thanks

fixed thanks

9:59 am

Michael Garthwaite

its a speed in rpm. updated function to convert target rate in ml/min to target speed in rpm and use that for the difference.

its a speed in rpm. updated function to convert target rate in ml/min to target speed in rpm and use that for the difference.

9:58 am

Michael Garthwaite

fixed thanks

fixed thanks

9:57 am

Michael Garthwaite

this is a set speed in rpm

this is a set speed in rpm

9:57 am

Michael Garthwaite

this is speed in rpm

this is speed in rpm

9:57 am

Michael Garthwaite

fixed thanks

fixed thanks

9:57 am

Michael Garthwaite

fixed thanks

fixed thanks

9:57 am

Michael Garthwaite

fixed thanks

fixed thanks

9:43 am

Sean Nash

Change speed to rate.

Change speed to rate.

9:43 am

Sean Nash

pumpTargetRate

pumpTargetRate

9:43 am

Sean Nash

Change function name to say rate instead of speed

Change function name to say rate instead of speed

9:42 am

Sean Nash

Is this a rate too?

Is this a rate too?

9:42 am

Sean Nash

Is this a rate too?

Is this a rate too?

9:40 am

Sean Nash

Change speed to rate.

Change speed to rate.

9:40 am

Sean Nash

Is currentPumpSpeed actually a rate?

Is currentPumpSpeed actually a rate?

9:38 am

Sean Nash

Is this comment correct? Seems like we're stopping pump if requested to turn off.

Is this comment correct? Seems like we're stopping pump if requested to turn off.

9:36 am

Sean Nash

Is currentPumpSpeed also a rate? If not, we're subtracting a speed from a rate which doesn't make much sense.

Is currentPumpSpeed also a rate? If not, we're subtracting a speed from a rate which doesn't make much sense.

9:34 am

Sean Nash

Align comment.

Align comment.

9:34 am

Sean Nash

Align comment. Change "speed" to "rate in comment too. Also, I don't see you initializing these rates.

Align comment. Change "speed" to "rate in comment too.
Also, I don't see you initializing these rates.