dgfirmware / firmware

App

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
Friday 25 Feb 2022
2:28 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Removed blank lines

Options
    • -2
    • +0
    ./Controllers/ConductivitySensors.h
    • -4
    • +2
    ./Modes/ModeFill.c
2:24 pm

Sean Nash

Looks like this needs to be addressed.

Looks like this needs to be addressed.

2:20 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Removed 5 secs persistent conductivity test SRSDG 558 & SRSDG 630

Options
    • -7
    • +0
    ./Controllers/ConductivitySensors.h
    • -29
    • +3
    ./Modes/ModeFill.c
11:02 am

Sean Nash

Magic #.

Magic #.

10:58 am

Sean Nash

rpm should be in ().

rpm should be in ().

10:56 am

Sean Nash

Should we check that payloadLength is at least 12 bytes (sizeof(U32) x 3)?

Should we check that payloadLength is at least 12 bytes (sizeof(U32) x 3)?

10:42 am

Sean Nash

There are 2 functions in this module with this name.

There are 2 functions in this module with this name.

8:57 am

Sean Nash

Are these really common? Can they be moved to sys_selftest.c?

Are these really common? Can they be moved to sys_selftest.c?

8:57 am

Dara Navaei

DG-DEN-11928_DG Conductivity Update
DG-DEN-11928_DG Conductivity Update
8:53 am

Sean Nash

Add #ifndef RELEASE around these definitions and all other build switch related code.

Add #ifndef RELEASE around these definitions and all other build switch related code.

8:51 am

Sean Nash

Add #ifndef RELEASE around this structure.

Add #ifndef RELEASE around this structure.

8:51 am

Sean Nash

Add #ifndef RELEASE around this enum.

Add #ifndef RELEASE around this enum.

8:50 am

Sean Nash

Should be #ifndef RELEASE around this structure.

Should be #ifndef RELEASE around this structure.

8:50 am

Sean Nash

I think there should be a #ifndef RELEASE around these build switches.

I think there should be a #ifndef RELEASE around these build switches.

8:46 am

Sean Nash

Dara, I keep seeing this change in every code review - Compatibility.h is deleted and Compatible.h is created. Why do we keep seeing this?

Dara, I keep seeing this change in every code review - Compatibility.h is deleted and Compatible.h is created. Why do we keep seeing this?

Thursday 24 Feb 2022
7:48 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
7:48 pm

Michael Garthwaite

Merge branch 'DEN-12121-sw_dev-sprint-65-michael-garthwaite' into develop

Options
    • -0
    • +3
    ./Services/SystemCommMessages.c
7:48 pm

Michael Garthwaite

DEN-12121: fixed memcpy

Options
    • -0
    • +3
    ./Services/SystemCommMessages.c
7:40 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
7:40 pm

Michael Garthwaite

Merge branch 'DEN-12121-sw_dev-sprint-65-michael-garthwaite' into develop

Options
    • -3
    • +1
    ./Services/SystemCommMessages.c
7:40 pm

Michael Garthwaite

DEN-12121: DIAL-147 fixed payload size check.

Options
    • -3
    • +1
    ./Services/SystemCommMessages.c
7:25 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
7:24 pm

Michael Garthwaite

Merge branch 'DEN-12121-sw_dev-sprint-65-michael-garthwaite' into develop

Options
    • -0
    • +4
    ./Services/SystemComm.c
7:24 pm

Michael Garthwaite

DEN-12121: switch case for HD communication override. DIAL-147

Options
    • -0
    • +4
    ./Services/SystemComm.c
6:31 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
6:26 pm

Michael Garthwaite

Merge branch 'DEN-12121-sw_dev-sprint-65-michael-garthwaite' into develop

Options
    • -4
    • +50
    ./Services/SystemComm.c
    • -1
    • +2
    ./Services/SystemComm.h
    • -0
    • +31
    ./Services/SystemCommMessages.c
    • -0
    • +3
    ./Services/SystemCommMessages.h
6:03 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Updated deliver dialysate state

Options
    • -0
    • +5
    ./Controllers/ConductivitySensors.h
    • -29
    • +57
    ./Modes/ModeFill.c
3:36 pm

Michael Garthwaite

Does this need to be memset when we can initialize it with 0'd values above? (line 92)

Does this need to be memset when we can initialize it with 0'd values above? (line 92)

3:33 pm

Michael Garthwaite

Do we need an if for an always true condition? does this branch statement need to be uncommented?

Do we need an if for an always true condition? does this branch statement need to be uncommented?

3:25 pm

Michael Garthwaite

Shouldn't the override check be its own get function like other overidden properties? Ex: getUVReactorHealth() ?

Shouldn't the override check be its own get function like other overidden properties? Ex: getUVReactorHealth() ?