leahi-common

MsgDefs.h

Clone Tools

Stats

Commits this week: 10
Total Committers: 19
Last Commit: 07 Jul
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
Friday 10 Jul
3:33 pm

Vendor - TEL - Praneeth Bunne

it will be adjusted when I merge latest staging

it will be adjusted when I merge latest staging

3:31 pm

Vendor - TEL - Praneeth Bunne

not required anymore, forgot to remove

not required anymore, forgot to remove

3:21 pm

Sean Nash

I think a comment here is needed. Keep it and maybe mention rinseback alarm responses too.

I think a comment here is needed. Keep it and maybe mention rinseback alarm responses too.

3:20 pm

Sean Nash

Maybe this should be handled in OperationModes exec since it applies to all confirmations (not just alarm related ones).

Maybe this should be handled in OperationModes exec since it applies to all confirmations (not just alarm related ones).

3:19 pm

Sean Nash

There will be confirmations that are not handled here (e.g. power off). So we shouldn't fault in default - we should do nothing.

There will be confirmations that are not handled here (e.g. power off). So we shouldn't fault in default - we should do nothing.

3:17 pm

Sean Nash

Shutdown would be handled somewhere else (buttons.c probably). It's not related to alarms.

Shutdown would be handled somewhere else (buttons.c probably). It's not related to alarms.

3:09 pm

Sean Nash

Is this still possible since we're not returning to TxPaused from rinseback anymore?

Is this still possible since we're not returning to TxPaused from rinseback anymore?

11:17 am

Sean Nash

Is rinseback completed an indication that we've done a blood prime following a temporary break? If so, I guess this makes sense, but we have to do more than increase the UF volume. We have to adjus...

Is rinseback completed an indication that we've done a blood prime following a temporary break?
If so, I guess this makes sense, but we have to do more than increase the UF volume. We have to adjust the UF rate so that we will reach the new (larger) UF volume when the treatment completes.
We also need to add RB volume to UF volume in pre-tx when treatment parameters are given/validated (for the final rinseback which we will always have so we should just plan for it before treatment starts).

11:03 am

Sean Nash

What if multiple confirmations pending? Which ID will this function return?

What if multiple confirmations pending? Which ID will this function return?

10:54 am

Sean Nash

Align comments. Should substitution pump data be moved down to 0xBA instead of commented out?

Align comments.
Should substitution pump data be moved down to 0xBA instead of commented out?

Thursday 09 Jul
4:22 pm

Vendor - TEL - Praneeth Bunne and Sean Nash

1:33 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-4040_Rinseback - FW Implementation - 3/10:
LEAHI-TD-FIRMWARE-LDT-4040_Rinseback - FW Implementation - 3/10:
9:48 am

Vendor - TEL - Vijay Pamula

Tuesday 07 Jul
3:57 pm

Sean Nash , Vendor - TEL - Praneeth Bunne , Zoltan Miskolci and Raghu Kallala

3:22 pm

Vendor - TEL - Vijay Pamula

LDT-4832:Addressed Review Comments

Options
    • -1
    • +1
    /MsgDefs.h
3:16 pm

Sean Nash

3:16 pm

Sean Nash

3:16 pm

Vendor - TEL - Praneeth Bunne

remove spaces before =

remove spaces before =

2:37 pm

Sean Nash

11:42 am

Vendor - TEL - Praneeth Bunne

change accordingly

change accordingly

11:42 am

Vendor - TEL - Praneeth Bunne

Rename to AlarmTableResposnePayload

Rename to AlarmTableResposnePayload

10:42 am

Vendor - TEL - Vijay Pamula

Merge branch 'staging' into LDT-4832-fw-ui-message-discrepancy

Options
    • -2
    • +3
    /MsgDefs.h
Monday 06 Jul
1:56 pm

Sean Nash

1:07 pm

Sean Nash

I would keep blank line between library includes and regular includes.

I would keep blank line between library includes and regular includes.

1:05 pm

Sean Nash

We can't have 2 alarms with ID 196. The enum only has the empty saline bag alarm. Looks like alarm 197 is duplicate of this 196 alarm you've added. I think you should just remove this new alarm 196...

We can't have 2 alarms with ID 196. The enum only has the empty saline bag alarm. Looks like alarm 197 is duplicate of this 196 alarm you've added.
I think you should just remove this new alarm 196 that you've added here.

10:56 am

Vendor - TEL - Praneeth Bunne

10:10 am

Vinayakam Mani

Friday 03 Jul
1:33 pm

Dara Navaei

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

Options
    • -2
    • +2
    /MsgDefs.h
  2. … 2 more files in changeset.
Thursday 02 Jul
3:32 pm

Sean Nash

test instead of handle

test instead of handle

3:06 pm

Vendor - TEL - Jashwant Gantyada and Sean Nash