dgfirmware / firmware / App

Services

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
Last Commit: 10 Jun
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 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.

Wednesday 19 May 2021
5:15 pm

Sean Nash

Should we move this enum to separate .h file?

Should we move this enum to separate .h file?

3:35 pm

pmontazemi  (deleted user)

Place TRUE as first argument.

Place TRUE as first argument.

3:29 pm

pmontazemi  (deleted user)

If we align, we align everything, otherwise, no extra spaces between variables and the "=" sign.

If we align, we align everything, otherwise, no extra spaces between variables and the "=" sign.

3:28 pm

pmontazemi  (deleted user)

Align comment.

Align comment.

10:38 am

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Alarm when firmware image integrity check failed

Options
    • -1
    • +12
    ./Integrity.c
Tuesday 18 May 2021
10:07 am

qnguyen  (deleted user)

DG-DEN-8103_DG HD Dev Alarm Design
DG-DEN-8103_DG HD Dev Alarm Design
Saturday 15 May 2021
9:51 am

Dara Navaei

I thought we decided to pass the broadcast data as a reference. Did we change that?

I thought we decided to pass the broadcast data as a reference. Did we change that?

Friday 14 May 2021
3:20 pm

qnguyen  (deleted user)

Removed casting.

Removed casting.

3:18 pm

qnguyen  (deleted user)

The macro already cast the data to U32.

The macro already cast the data to U32.

1:43 pm

Sean Nash

Should maybe cast here though.

Should maybe cast here though.

1:43 pm

Sean Nash

Why cast here?

Why cast here?

11:51 am

Dara Navaei

DEN-8030 added more logic to check the load cells value during tare

Options
    • -1
    • +1
    ./Reservoirs.c
  2. … 1 more file in changeset.
Thursday 13 May 2021
5:59 pm

Sean Nash

DEN-7605: Pulled in V3 valve changes from chem disinfect branch.

Options
    • -2
    • +10
    ./Reservoirs.c
  2. … 7 more files in changeset.