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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 24 Mar 2021
11:49 am

Quang Nguyen <qnguyen@diality.com>

DEN-7091: Added treatment start message handler

Options
    • -89
    • +5
    /firmware/App/Modes/Dialysis.c
    • -7
    • +5
    /firmware/App/Modes/ModePreTreat.c
    • -1
    • +1
    /firmware/App/Modes/ModePreTreat.h
    • -6
    • +16
    /firmware/App/Modes/ModeStandby.c
    • -1
    • +1
    /firmware/App/Modes/ModeStandby.h
    • -1
    • +4
    /firmware/App/Modes/ModeTreatmentParams.c
    • -1
    • +1
    /firmware/App/Modes/OperationModes.c
    • -1
    • +1
    /firmware/App/Modes/Prime.h
    • -0
    • +1
    /firmware/App/Modes/SelfTests.c
    • -3
    • +7
    /firmware/App/Services/SystemComm.c
    • -30
    • +43
    /firmware/App/Services/SystemCommMessages.c
    • -4
    • +10
    /firmware/App/Services/SystemCommMessages.h
10:36 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:35 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:27 am

qnguyen  (deleted user)

This was temporary before a blood leak driver. Delete if not needed.

This was temporary before a blood leak driver. Delete if not needed.

10:27 am

qnguyen  (deleted user)

This was temporary before a blood leak driver. Delete if not needed.

This was temporary before a blood leak driver. Delete if not needed.

10:25 am

qnguyen  (deleted user)

Why this one is deleted?

Why this one is deleted?

10:22 am

Sean Nash

Do we need a get function for determining if/when zero is done? And another for determining if self-test passed/failed?

Do we need a get function for determining if/when zero is done? And another for determining if self-test passed/failed?

10:22 am

qnguyen  (deleted user)

Remove this. No need to add if there are no parameters.

Remove this. No need to add if there are no parameters.

10:21 am

Sean Nash

Alarm if in patient connected mode/sub-mode, fault if not.

Alarm if in patient connected mode/sub-mode, fault if not.

9:52 am

Sean Nash

Check should only be done if not doing self-test. If doing self-test, a different (opposite) check should be done (if blood not detected within TBD ms, fault).

Check should only be done if not doing self-test. If doing self-test, a different (opposite) check should be done (if blood not detected within TBD ms, fault).

9:46 am

Sean Nash

What about functions for 1) getting zero status, 2) setting sensor config (normal, self-test, zero)?

What about functions for 1) getting zero status, 2) setting sensor config (normal, self-test, zero)?

9:43 am

Sean Nash

Should be 0x1000? And shouldn't we add another mask for blood leak zero state (0x2000)?

Should be 0x1000?
And shouldn't we add another mask for blood leak zero state (0x2000)?

9:18 am

Sean Nash

We should consider adding Dialin options to zero sensor and initiate self-test.

We should consider adding Dialin options to zero sensor and initiate self-test.

9:17 am

Sean Nash

Move both leak monitors to monitors section above.

Move both leak monitors to monitors section above.

9:17 am

Sean Nash

These need to be called from switch statement at bottom of SystemComm.c.

These need to be called from switch statement at bottom of SystemComm.c.

9:11 am

Dara Navaei

DEN-5980 addressed code review comments

Options
    • -1
    • +1
    /firmware/App/Controllers/DialInFlow.c
    • -1
    • +1
    /firmware/App/Services/SystemComm.c
9:10 am

Dara Navaei

Done.

Done.

8:53 am

Dara Navaei

Done.

Done.

8:48 am

Dara Navaei

Done.

Done.

8:47 am

Dara Navaei

Done.

Done.

Tuesday 23 Mar 2021
4:58 pm

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Resolved merge issues

Options
    • -23
    • +0
    /firmware/App/Modes/ModePreTreat.c
    • -269
    • +0
    /firmware/App/Modes/SelfTests.c
    • -1
    • +0
    /firmware/App/Modes/SelfTests.h
4:09 pm

Peman Montazemi <pmontazemi@diality.com>

DEN-7395: Added handle test implementation for Blood Leak Detector driver overrides

Options
    • -0
    • +6
    /firmware/App/Services/SystemCommMessages.h
4:05 pm

pmontazemi  (deleted user)

HD-DEN-7395_HD Blood Leak Detector
HD-DEN-7395_HD Blood Leak Detector
2:47 pm

Sean Nash

Merge branch 'DEN-7117-hd_dev-hd-heparin-delivery' into develop

Options
    • -8
    • +1
    /firmware/App/Controllers/BloodFlow.c
    • -2
    • +2
    /firmware/App/Controllers/DGInterface.c
    • -1
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -1
    • +0
    /firmware/App/HDCommon.h
    • -2
    • +2
    /firmware/App/Modes/Prime.c
2:47 pm

Sean Nash

DEN-7117: Addressed code review feedback.

Options
    • -2
    • +2
    /firmware/App/Controllers/DGInterface.c
    • -1
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -1
    • +1
    /firmware/App/Modes/Dialysis.c
    • -2
    • +2
    /firmware/App/Modes/Prime.c
2:05 pm

Sean Nash

Done.

Done.

2:02 pm

Sean Nash

It is volts. Everywhere else we use abbreviated units of measure as suffix (e.g. _ML).

It is volts. Everywhere else we use abbreviated units of measure as suffix (e.g. _ML).