hdfirmware

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
Last Commit: 19 Nov 25
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 08 Apr 2021
3:00 pm

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Lumped Blood Leak Detector status and state into one published data

Options
    • -190
    • +52
    /firmware/App/Controllers/BloodLeak.c
    • -10
    • +5
    /firmware/App/Controllers/BloodLeak.h
    • -11
    • +4
    /firmware/App/Services/SystemComm.c
    • -108
    • +18
    /firmware/App/Services/SystemCommMessages.c
    • -15
    • +6
    /firmware/App/Services/SystemCommMessages.h
2:59 pm

pmontazemi  (deleted user)

Addressed.

Addressed.

1:22 pm

Sean Nash

Merge branch 'DEN-7605-hd_dg_dev-self-tests' into develop

Options
1:21 pm

Sean Nash

Fixed merge conflicts.

Options
    • -69
    • +7
    /firmware/App/Controllers/BloodFlow.c
    • -65
    • +6
    /firmware/App/Controllers/DialInFlow.c
    • -3
    • +10
    /firmware/App/Modes/ModePreTreat.c
    • -72
    • +97
    /firmware/App/Modes/SelfTests.c
10:41 am

Quang Nguyen <qnguyen@diality.com>

DEN-7347: Added build switch to skip empty reservoir check

Options
    • -0
    • +1
    /firmware/App/HDCommon.h
    • -0
    • +6
    /firmware/App/Modes/ModePostTreat.c
10:41 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:36 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

Sean Nash

Need to add state machine in this function. Call state handler functions from a switch statement.

Need to add state machine in this function. Call state handler functions from a switch statement.

10:31 am

Dara Navaei

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:31 am

Sean Nash

Preferred architecture is to just set flag here and then let your state machine see the flag and change the state and command the fpga when it next executes.

Preferred architecture is to just set flag here and then let your state machine see the flag and change the state and command the fpga when it next executes.

10:29 am

Sean Nash

Preferred architecture is to just set a request flag here and then let your state machine see the flag and change the state and command the FPGA action the next time it executes.

Preferred architecture is to just set a request flag here and then let your state machine see the flag and change the state and command the FPGA action the next time it executes.

10:28 am

qnguyen  (deleted user)

getCurrentOperationMode() returns the current operation mode (standby, pre-treatment, treatment, post-treatment, etc.), not the sub-mode.

getCurrentOperationMode() returns the current operation mode (standby, pre-treatment, treatment, post-treatment, etc.), not the sub-mode.

10:25 am

pmontazemi  (deleted user)

One is for the Blood Leak Detector status (blood leaked or blood not leaked), and the other one is for the Blood Leak Detector state of its state machine. This is just in case we wanted to have ind...

One is for the Blood Leak Detector status (blood leaked or blood not leaked), and the other one is for the Blood Leak Detector state of its state machine. This is just in case we wanted to have independent control over these time constants.

10:23 am

Sean Nash

But they could be packaged together into once blood leak data broadcast message.

But they could be packaged together into once blood leak data broadcast message.

10:20 am

Sean Nash

This is more of a request to run a self test by a mode (probably pre-treatment mode), so I think boolean is appropriate to indicate request was accepted.

This is more of a request to run a self test by a mode (probably pre-treatment mode), so I think boolean is appropriate to indicate request was accepted.

10:18 am

qnguyen  (deleted user)

Temperature values have been extracted out from treatment parameters in HD-DEN-7091.

Temperature values have been extracted out from treatment parameters in HD-DEN-7091.

10:15 am

Sean Nash

What is the difference between this interval override and the one below? If monitor is broadcasting 2 different messages, why? Recommend combining published blood leak data into one message.

What is the difference between this interval override and the one below? If monitor is broadcasting 2 different messages, why? Recommend combining published blood leak data into one message.

10:08 am

pmontazemi  (deleted user)

They publish two different values that can be set independently of each other.

They publish two different values that can be set independently of each other.

9:58 am

Dara Navaei

Can these two publish functions be lumped together?

Can these two publish functions be lumped together?

9:57 am

Dara Navaei

Usually the self tests return SELF_TEST_STATUS_T. Why does this function return a BOOL instead?

Usually the self tests return SELF_TEST_STATUS_T. Why does this function return a BOOL instead?

9:51 am

pmontazemi  (deleted user)

[~qnguyen] Please address.

[~qnguyen] Please address.

9:42 am

Dara Navaei

Recommend adding #defines for the temperature values.

Recommend adding #defines for the temperature values.

2:07 am

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Corrected naming for Blood Leak Detector driver

Options
    • -5
    • +5
    /firmware/App/Controllers/BloodLeak.c
    • -3
    • +3
    /firmware/App/Controllers/BloodLeak.h
    • -2
    • +2
    /firmware/App/Services/FPGA.c
    • -2
    • +31
    /firmware/App/Services/SystemCommMessages.c
    • -1
    • +1
    /firmware/App/Services/SystemCommMessages.h
1:55 am

pmontazemi  (deleted user)

Addressed.

Addressed.

1:54 am

pmontazemi  (deleted user)

Addressed.

Addressed.

1:52 am

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Fixed namings for Blood Leak Detector driver

Options
    • -0
    • +20
    /firmware/App/Controllers/BloodLeak.c
    • -0
    • +1
    /firmware/App/Controllers/BloodLeak.h
1:34 am

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Added messaging for Blood Leak Detector state machine

Options
    • -38
    • +206
    /firmware/App/Controllers/BloodLeak.c
    • -6
    • +22
    /firmware/App/Controllers/BloodLeak.h
    • -1
    • +1
    /firmware/App/Services/FPGA.c
    • -2
    • +9
    /firmware/App/Services/SystemComm.c
    • -15
    • +79
    /firmware/App/Services/SystemCommMessages.c
    • -3
    • +12
    /firmware/App/Services/SystemCommMessages.h
Wednesday 07 Apr 2021
5:20 pm

Sean Nash

DEN-7605: Added dependency check for initial UF volume setting.

Options
    • -2
    • +0
    /firmware/App/Modes/ModeTreatment.c
    • -1
    • +33
    /firmware/App/Modes/ModeTreatmentParams.c
    • -1
    • +3
    /firmware/App/Modes/ModeTreatmentParams.h
1:13 pm

qnguyen  (deleted user)

Removed.

Removed.

12:42 pm

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Renamed Blood Leak State to Blood Leak Status to allow naming for Blood Leak state machine

Options
    • -49
    • +49
    /firmware/App/Controllers/BloodLeak.c
    • -5
    • +5
    /firmware/App/Controllers/BloodLeak.h
    • -2
    • +2
    /firmware/App/Services/SystemComm.c
    • -11
    • +11
    /firmware/App/Services/SystemCommMessages.c
    • -4
    • +4
    /firmware/App/Services/SystemCommMessages.h