dgfirmware / firmware / App

Services

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
Saturday 29 May 2021
12:58 am

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Moved integrity module to fwcommon

Options
    • -1
    • +0
    ./AlarmMgmtSWFaults.h
    • -3
    • +3
    ./FPGA.c
    • -135
    • +0
    ./Integrity.c
    • -40
    • +0
    ./Integrity.h
  5. … 1 more file in changeset.
Friday 28 May 2021
4:39 pm

qnguyen  (deleted user)

Done.

Done.

4:33 pm

qnguyen  (deleted user)

Noe said they updated the ID and this is the new ID for DG FPGA. The FPGA test will fail without this change.

Noe said they updated the ID and this is the new ID for DG FPGA.
The FPGA test will fail without this change.

3:20 pm

Sean Nash

Should mention somewhere that timer is counting ms.

Should mention somewhere that timer is counting ms.

3:19 pm

Sean Nash

Why did this change?

Why did this change?

3:18 pm

Sean Nash

Is there a reason why this module isn't in fwcommon? And if it can't be common for some reason, should this module (or group name) be named DGIntegrity?

Is there a reason why this module isn't in fwcommon? And if it can't be common for some reason, should this module (or group name) be named DGIntegrity?

3:18 pm

Sean Nash

Do we need to support more than one 32-bit CRC algorithm in firmware? If not, do we need this wrapper function?

Do we need to support more than one 32-bit CRC algorithm in firmware? If not, do we need this wrapper function?

Thursday 27 May 2021
10:34 am

qnguyen  (deleted user)

Done.

Done.

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

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:31 am

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:30 am

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Wednesday 26 May 2021
10:55 am

Sean Nash

DEN-7605: Addressed code review feedback.

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

Sean Nash

DEN-7605: Addressed code review feedback.

Options
    • -2
    • +2
    ./Reservoirs.c
  2. … 1 more file in changeset.
9:35 am

Sean Nash

Fixed.

Fixed.

9:06 am

Sean Nash

Yes. Fixed.

Yes. Fixed.

9:03 am

qnguyen  (deleted user)

Consider re-word to something more general. This function might be used by ModeHeatDisinfect to tare on an active reservoir.

Consider re-word to something more general. This function might be used by ModeHeatDisinfect to tare on an active reservoir.

8:58 am

qnguyen  (deleted user)

Should this be in interval count?

Should this be in interval count?

Tuesday 25 May 2021
1:12 pm

Sean Nash

Fixed merge conflicts.

Options
    • -7
    • +33
    ./Reservoirs.c
    • -3
    • +5
    ./Reservoirs.h
    • -4
    • +8
    ./SystemComm.c
    • -0
    • +106
    ./SystemCommMessages.c
    • -0
    • +9
    ./SystemCommMessages.h
  6. … 3 more files in changeset.
12:59 pm

Sean Nash

DEN-7605: Added support for UI clock sync.

Options
    • -4
    • +7
    ./SystemComm.c
    • -0
    • +45
    ./SystemCommMessages.c
    • -0
    • +3
    ./SystemCommMessages.h
Monday 24 May 2021
11:20 pm

Sean Nash

DEN-7605: Added POST messaging.

Options
    • -0
    • +61
    ./SystemCommMessages.c
    • -0
    • +6
    ./SystemCommMessages.h
  3. … 1 more file in changeset.
3:00 pm

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Added safety check to fill using flow meter

Options
    • -1
    • +0
    ./Reservoirs.c
    • -0
    • +2
    ./Reservoirs.h
  3. … 1 more file in changeset.
Friday 21 May 2021
2:35 pm

Sean Nash

DEN-7605: Added tare state to drain mode to allow time for reservoir to settle first.

Options
    • -7
    • +33
    ./Reservoirs.c
    • -1
    • +3
    ./Reservoirs.h
  3. … 1 more file in changeset.
10:42 am

Dara Navaei

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:20 am

Sean Nash

Fixed.

Fixed.

10:20 am

Sean Nash

Fixed.

Fixed.

Thursday 20 May 2021
12:18 pm

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Imposed maximum crc size

Options
    • -101
    • +2
    ./AlarmMgmt.h
    • -0
    • +130
    ./AlarmMgmtSWFaults.h
    • -2
    • +14
    ./FPGA.c
    • -0
    • +2
    ./FPGA.h
    • -25
    • +54
    ./Integrity.c
11:26 am

qnguyen  (deleted user)

The only preference is to put TRUE first if it is == such as below.

The only preference is to put TRUE first if it is == such as below.

8:58 am

Sean Nash

I have no preference on that. Data is temporary either way. By reference is a little easier on the stack.

I have no preference on that. Data is temporary either way. By reference is a little easier on the stack.