leahi-common

Clone Tools

Stats

Commits this week: 13
Total Committers: 19
Last Commit: 16:33
Commits this week: 13

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

Vendor - TEL - Praneeth Bunne

LDT-1886: Resolve merge conflicts

Options
    • -5
    • +61
    /AlarmDefs.h
    • -22
    • +25
    /TDDefs.h
8 hrs ago

Sean Nash

8 hrs ago

Sean Nash

Align comment.

Align comment.

8 hrs ago

Sean Nash

Keep 2 blank lines above banner.

Keep 2 blank lines above banner.

8 hrs ago

Sean Nash

Should this line be un-commented?

Should this line be un-commented?

8 hrs ago

Sean Nash

So we are going to fail this test until calibration record is ready?

So we are going to fail this test until calibration record is ready?

8 hrs ago

Sean Nash

In Leahi, we want to do all initialization in the init function instead of here at declaration (see Bubble.c for example).

In Leahi, we want to do all initialization in the init function instead of here at declaration (see Bubble.c for example).

13 hrs ago

Sean Nash

Rename similar to ADDR definitions below. these are not commands.

Rename similar to ADDR definitions below. these are not commands.

13 hrs ago

Sean Nash

Combine charger and pack data into one payload with one broadcast message.

Combine charger and pack data into one payload with one broadcast message.

14 hrs ago

Sean Nash

Move these 3 override messages (should end in OVERRIDE_REQUEST) to TD Dialin section below and assign an ID in the 0x8... range.

Move these 3 override messages (should end in OVERRIDE_REQUEST) to TD Dialin section below and assign an ID in the 0x8... range.

14 hrs ago

Sean Nash

I think we only need one data message for both charger and battery pack data.

I think we only need one data message for both charger and battery pack data.

14 hrs ago

Sean Nash

Remove.

Remove.

14 hrs ago

Sean Nash

uncomment.

uncomment.

Monday 18 May
4:28 pm

suresh

LDT-4072: Fix battery alarm priority rank and add message IDs

Options
    • -4
    • +6
    /AlarmDefs.h
    • -2
    • +5
    /MsgDefs.h
12:05 pm

Vendor - TEL - Arpita Srivastava

LDT-3505: Updated force sensor calibration message id

Options
    • -2
    • +2
    /MsgDefs.h
9:45 am

Vendor - TEL - Arpita Srivastava

please add bpLeftHomeTimerCounter to inputs and outputs.

please add bpLeftHomeTimerCounter to inputs and outputs.

9:44 am

Vendor - TEL - Arpita Srivastava

please add bpLeftHomeTimerCounter to inputs and outputs.

please add bpLeftHomeTimerCounter to inputs and outputs.

Sunday 17 May
11:46 pm

Sean Nash

Why was this function header changed? I think it was correct before.

Why was this function header changed? I think it was correct before.

11:37 pm

Sean Nash

If these messages are for change to UF volume during treatment, the message ID should have the word TREATMENT in it to distinguish from the original UF volume message that comes in pre-treatment.

If these messages are for change to UF volume during treatment, the message ID should have the word TREATMENT in it to distinguish from the original UF volume message that comes in pre-treatment.

11:34 pm

Sean Nash

It is weird that we're responding so far from request. It is not clear to me that Dialin has requested anything at this point.

It is weird that we're responding so far from request. It is not clear to me that Dialin has requested anything at this point.

11:26 pm

Sean Nash

Details looks weird. Where is inputs and outputs like other function headers?

Details looks weird. Where is inputs and outputs like other function headers?

11:23 pm

Sean Nash

Now it's backward. You have to change the ENABLE to DISABLE too.

Now it's backward. You have to change the ENABLE to DISABLE too.

11:19 pm

Sean Nash

Add a blank line after banner.

Add a blank line after banner.

11:19 pm

Sean Nash

Please restore before you merge to staging.

Please restore before you merge to staging.

11:16 pm

Sean Nash

If I'm reading this right, we're saying that if bad CRC alarm is NOT enabled, initialize the alarm? That seems backward.

If I'm reading this right, we're saying that if bad CRC alarm is NOT enabled, initialize the alarm? That seems backward.

11:14 pm

Sean Nash

Same comments as ENABLE_ALARM_1 above.

Same comments as ENABLE_ALARM_1 above.

11:13 pm

Sean Nash

Same comments as ENABLE_ALARM_1 above.

Same comments as ENABLE_ALARM_1 above.

11:13 pm

Sean Nash

This definitions has a terrible name (I have no idea what it does) and I think it should be reversed (i.e. alarms are disabled when defined so that a release build will have alarms enabled).

This definitions has a terrible name (I have no idea what it does) and I think it should be reversed (i.e. alarms are disabled when defined so that a release build will have alarms enabled).

Friday 15 May
6:12 pm

Sean Nash

Call the init for the battery driver here.

Call the init for the battery driver here.

6:11 pm

Sean Nash

INVALID_BATTERY

INVALID_BATTERY