hdfirmware

firmware

Clone Tools

Stats

Commits this week: 14
Total Committers: 17
Last Commit: 10 Jun
Commits this week: 14

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
Monday 17 May 2021
10:56 am

Peman Montazemi <pmontazemi@diality.com>

DEN-7860: Addressed code review comments

Options
    • -1
    • +4
    ./App/Controllers/Bubble.c
10:56 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:55 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:55 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:54 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:54 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:53 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:53 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:53 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:53 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:52 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:50 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:49 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:47 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:47 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:41 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:41 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:40 am

pmontazemi  (deleted user)

We plan on doing self-test as part of priming, so we confirm the bubble detector reads air on a new cartridge, then we check that we read fluid when we have completed priming. I need to add these c...

We plan on doing self-test as part of priming, so we confirm the bubble detector reads air on a new cartridge, then we check that we read fluid when we have completed priming. I need to add these checks. The air bubble detector alarm, however, should only be active during treatment (includes blood prime and rinseback), and outside of that we just monitor the detectors but never alarm. Therefore, we do not need to do the self-test in POST and it is fine to keep handleInitBubbleState(...) as is.

Note: No code change to handleInitBubbleState(...).

10:39 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:22 am

Sean Nash

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

Options
    • -1
    • +1
    ./App/HDCommon.h
10:21 am

Sean Nash

DEN-7605: disable syringe pump speed check if DISABLE_SYRINGE_ALARMS is defined.

Options
    • -2
    • +4
    ./App/Controllers/SyringePump.c
    • -1
    • +1
    ./App/HDCommon.h
10:06 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:06 am

pmontazemi  (deleted user)

Addressed.

Addressed.

10:00 am

pmontazemi  (deleted user)

Compiler gives warnings with U32 casting, left code as is.

Compiler gives warnings with U32 casting, left code as is.

9:59 am

Peman Montazemi <pmontazemi@diality.com>

DEN-7860: Addressed code review comments

Options
    • -0
    • +12
    ./App/Services/SystemComm.c
9:59 am

pmontazemi  (deleted user)

Addressed.

Addressed.

9:48 am

Peman Montazemi <pmontazemi@diality.com>

DEN-7860: Addressed code review comments

Options
    • -44
    • +43
    ./App/Controllers/Bubble.c
    • -7
    • +5
    ./App/Services/FPGA.c
9:47 am

pmontazemi  (deleted user)

Addressed.

Addressed.

9:43 am

pmontazemi  (deleted user)

Removed ADA/ADV bubble state test masks as there are no separate FPGA registers that output these variables.

Removed ADA/ADV bubble state test masks as there are no separate FPGA registers that output these variables.

9:40 am

pmontazemi  (deleted user)

Removed ADA/ADV bubble state test masks.

Removed ADA/ADV bubble state test masks.