hdfirmware / firmware / App

Services

Clone Tools

Stats

Commits this week: 0
Total Committers: 16
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 10 Jun 2021
9:40 pm

Dara Navaei

DEN-8030 added messages and code to publish the status of the disinfects from DG

Options
    • -0
    • +12
    ./SystemComm.c
    • -0
    • +224
    ./SystemCommMessages.c
    • -0
    • +28
    ./SystemCommMessages.h
  4. … 7 more files in changeset.
3:22 pm

Sean Nash

3:04 pm

qnguyen  (deleted user) , Dara Navaei and pmontazemi  (deleted user)

10:31 am

Sean Nash

DEN-7605: Local merge with staging.

Options
    • -2
    • +7
    ./AlarmMgmt.h
    • -53
    • +156
    ./FPGA.c
    • -3
    • +11
    ./FPGA.h
    • -1
    • +37
    ./SystemComm.c
    • -10
    • +374
    ./SystemCommMessages.c
    • -6
    • +51
    ./SystemCommMessages.h
  7. … 5 more files in changeset.
Monday 07 Jun 2021
1:31 pm

Sean Nash

Fixed merge conflicts.

Options
    • -7
    • +9
    ./AlarmMgmt.c
    • -2
    • +2
    ./SystemCommMessages.c
  3. … 1 more file in changeset.
10:35 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:34 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:34 am

qnguyen  (deleted user)

Per Dara Navaei, we will need to implement this functions.

Per Dara Navaei, we will need to implement this functions.

10:34 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:34 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.

9:34 am

Sean Nash

DEN-7605: pass alarm audio test until test properly implemented.

Options
    • -7
    • +9
    ./AlarmMgmt.c
    • -1
    • +1
    ./FPGA.c
    • -2
    • +2
    ./SystemCommMessages.c
9:27 am

Sean Nash

Good point. Test will need to be longer than 50ms anyway - FPGA ramps the sound and there is a ~250ms time constant in the electronics. But I will want the sound to at least get started immediately.

Good point. Test will need to be longer than 50ms anyway - FPGA ramps the sound and there is a ~250ms time constant in the electronics. But I will want the sound to at least get started immediately.

Friday 04 Jun 2021
12:52 pm

Sean Nash

Disabled pump checks.

Options
    • -1
    • +1
    ./FPGA.c
  2. … 2 more files in changeset.
10:31 am

qnguyen  (deleted user)

Should we call setAlarmAudio here? This test is running at 50 ms while the setAlarmAudio is running at 250 ms. The alarm audio might not be set when we switch to else part.

Should we call setAlarmAudio here?
This test is running at 50 ms while the setAlarmAudio is running at 250 ms. The alarm audio might not be set when we switch to else part.

10:19 am

qnguyen  (deleted user)

The flag is latched by using OR with the local updated noNewTreatment status.

The flag is latched by using OR with the local updated noNewTreatment status.

10:15 am

Sean Nash

But now in order to use the status flag, I think we can't allow that flag to go from TRUE to FALSE - need to latch it.

But now in order to use the status flag, I think we can't allow that flag to go from TRUE to FALSE - need to latch it.

Thursday 03 Jun 2021
6:05 pm

Sean Nash

Fixed build flags.

Options
    • -2
    • +2
    ./SystemCommMessages.c
  2. … 1 more file in changeset.
10:55 am

qnguyen  (deleted user)

Removed this extra flag and used alarm status noNewTreatment flag.

Removed this extra flag and used alarm status noNewTreatment flag.

10:54 am

qnguyen  (deleted user)

Removed this extra flag.

Removed this extra flag.

10:52 am

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Addressed code review

Options
    • -15
    • +7
    ./AlarmMgmt.c
    • -1
    • +1
    ./AlarmMgmt.h
  3. … 1 more file in changeset.
10:51 am

qnguyen  (deleted user)

Done.

Done.

10:33 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:27 am

qnguyen  (deleted user)

Yes, and we checked for DG restarted (after it has started by HD) in DGInterface. Sample water sub-mode has to check before DG started.

Yes, and we checked for DG restarted (after it has started by HD) in DGInterface.
Sample water sub-mode has to check before DG started.

9:57 am

Sean Nash

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

Options
    • -5
    • +57
    ./AlarmMgmt.c
    • -1
    • +2
    ./AlarmMgmt.h
    • -10
    • +12
    ./FPGA.c
    • -1
    • +1
    ./FPGA.h
  5. … 2 more files in changeset.
9:55 am

Sean Nash

DEN-7605: Added POST test for alarm audio.

Options
    • -5
    • +57
    ./AlarmMgmt.c
    • -1
    • +2
    ./AlarmMgmt.h
    • -10
    • +12
    ./FPGA.c
    • -1
    • +1
    ./FPGA.h
  5. … 3 more files in changeset.
9:07 am

Sean Nash

Looks like you are latching the noEndTreatment status. Maybe we can have that status be latching so we don't need two flags.

Looks like you are latching the noEndTreatment status. Maybe we can have that status be latching so we don't need two flags.

9:03 am

Sean Nash

Is this different than alarmStatus.noNewTreatment?

Is this different than alarmStatus.noNewTreatment?

9:01 am

Sean Nash

Shouldn't this be checked more broadly (not just in water sample sub-mode)?

Shouldn't this be checked more broadly (not just in water sample sub-mode)?