hdfirmware / firmware / App / Services

SystemComm.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 15
Last Commit: 19 Nov
Commits this week: 0

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 16 Mar 2023
8:22 pm

Dara Navaei

DEN-14763 added commands to be able to issue hard stop the HD pumps from Dialin

Options
    • -0
    • +13
    ./SystemComm.c
  2. … 4 more files in changeset.
1:42 pm

Michael Garthwaite

Merge branch 'DEN-14921-machine-alarm-30-on-dvt013' into develop

Options
    • -2
    • +2
    ./SystemComm.c
1:36 pm

Michael Garthwaite

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:36 pm

Michael Garthwaite

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

11:59 am

Sean Nash

Fixed.

Fixed.

11:58 am

Sean Nash

Fixed.

Fixed.

11:51 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

11:46 am

Michael Garthwaite

Do we still need the TODO?

Do we still need the TODO?

11:41 am

Michael Garthwaite

Please add these event types to your dialin CR.

Please add these event types to your dialin CR.

10:56 am

Sean Nash

Was added, then moved to another location. Fixed.

Was added, then moved to another location.
Fixed.

10:36 am

Sean Nash

Merge branch 'DEN-14918-pba-pbo-and-ob-not-available-on-debug-screen' into develop

Options
    • -10
    • +10
    ./SystemComm.c
Monday 13 Mar 2023
1:52 pm

Sean Nash

DEN-14888: Implemented user confirmation of treatment end response to alarm.

Options
    • -2
    • +0
    ./SystemComm.c
  2. … 5 more files in changeset.
Sunday 12 Mar 2023
10:01 pm

wbracken  (deleted user)

Don't see timeReservoirInUF being used?

Don't see timeReservoirInUF being used?

Friday 10 Mar 2023
4:01 pm

Sean Nash

3:59 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./SystemComm.c
3:58 pm

Dara Navaei and wbracken  (deleted user)

3:41 pm

Dara Navaei

HD-DEN-14918_Pba Pbo And OB Not Available ON Debug Screen
HD-DEN-14918_Pba Pbo And OB Not Available ON Debug Screen
3:14 pm

Sean Nash

DEN-14918: Fixed issue with alarm 91 when DG comm lost outside of treatment workflow.

Options
    • -10
    • +10
    ./SystemComm.c
1:20 pm

wbracken  (deleted user)

Isn't deleted statement correct?

Isn't deleted statement correct?

1:19 pm

wbracken  (deleted user)

Isn't deleted statement correct?

Isn't deleted statement correct?

1:18 pm

wbracken  (deleted user)

Isn't deleted statement correct?

Isn't deleted statement correct?

1:17 pm

wbracken  (deleted user)

Isn't delete statement correct?

Isn't delete statement correct?

10:19 am

Sean Nash

I think that's true in calls to serialize function, but I don't think this function wants this kind of flag (i.e. it's not looking for whether to require ACK or not - it wants to know if we are ACK...

I think that's true in calls to serialize function, but I don't think this function wants this kind of flag (i.e. it's not looking for whether to require ACK or not - it wants to know if we are ACKing (TRUE) or NAKing (FALSE) this command). Here, we are NAKing the command, so FALSE was appropriate.

10:16 am

Sean Nash

Has DG check-in msg ID been made available for re-purpose? RESOLVED in CODE WALKTHROUGH.

Has DG check-in msg ID been made available for re-purpose?
RESOLVED in CODE WALKTHROUGH.

10:14 am

Michael Garthwaite

DG no longer sends this message. We determine DG check-in by receiving the DG op mode command.

DG no longer sends this message. We determine DG check-in by receiving the DG op mode command.

10:01 am

Dara Navaei

HD-DEN-14888_Unable TO Trigger Alarm 47 Ultrafiltration Rate IS Too High Error During Treatment
HD-DEN-14888_Unable TO Trigger Alarm 47 Ultrafiltration Rate IS Too High Error During Treatment
9:48 am

Sean Nash

Why removed?

Why removed?

Thursday 09 Mar 2023
11:35 am

wbracken  (deleted user)

Change cmd to prompt

Change cmd to prompt

11:08 am

Michael Garthwaite

How do we wish to handle the output of the automated script? It currently does not handle conditional statements to determine ack requiredness. If set to false, the output is: 0x4A MSG_ID_UI_PATI...

How do we wish to handle the output of the automated script? It currently does not handle conditional statements to determine ack requiredness.

If set to false, the output is:

0x4A MSG_ID_UI_PATIENT_DISCONNECTION_CONFIRM hdfirmware response COMM_BUFFER_OUT_CAN_HD_2_UI ACK_REQUIRED PAYLOAD

which is incorrect to what the code is doing. We have 2 different implementations of calling sendAckResponseMsg()


1. sendAckResponseMsg( (MSG_ID_T)message->hdr.msgID, COMM_BUFFER_OUT_CAN_HD_2_UI, FALSE );

  1. if we dont meet an expected condition. Later in the handler we will call a response message with the response message calling:
    serializeMessage( msg, COMM_BUFFER_OUT_CAN_DG_2_HD, ACK_REQUIRED );


2. sendAckResponseMsg( (MSG_ID_T)message->hdr.msgID, COMM_BUFFER_OUT_CAN_HD_2_UI, result );

  1. a conditional based on?


Please review the FW_Message_list.csv within latest develop & staging builds to understand how the automated script is determining "ack requireness"

9:33 am

Michael Garthwaite

DEN-14689: removed dg checkin message as it is no longer needed.

Options
    • -4
    • +0
    ./SystemComm.c
  2. … 1 more file in changeset.