dgfirmware / firmware / App

Services

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
Last Commit: 19 Nov
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
Thursday 06 Oct 2022
9:20 am

Sean Nash

DEN-13834: Added additional override of load cell after calibration but before filter/tare.

Options
    • -6
    • +12
    ./SystemCommMessages.c
  2. … 2 more files in changeset.
Tuesday 04 Oct 2022
5:17 pm

Sean Nash

Do we still need this? If so, can we make a s/w config instead?

Do we still need this? If so, can we make a s/w config instead?

5:14 pm

Sean Nash

Is this resolved?

Is this resolved?

5:04 pm

Sean Nash

Why are we not checking payload length?

Why are we not checking payload length?

5:03 pm

Sean Nash

If this function is not yet needed, should we remove or at least put a #ifdef PHASE_1B build flag around it? Otherwise this is dead code right now.

If this function is not yet needed, should we remove or at least put a #ifdef PHASE_1B build flag around it? Otherwise this is dead code right now.

5:00 pm

Sean Nash

Why are we not enforcing payload length check?

Why are we not enforcing payload length check?

5:00 pm

Sean Nash

Can this declaration be moved to top of function?

Can this declaration be moved to top of function?

4:57 pm

Sean Nash

Why are we hiding this condition from Vectorcast?

Why are we hiding this condition from Vectorcast?

4:55 pm

Sean Nash

This comment should precede the if statement above.

This comment should precede the if statement above.

4:52 pm

Sean Nash

Is this 109?

Is this 109?

4:51 pm

Sean Nash

This looks like 104 (not 106). And we should be noting every 5th enum anyway.

This looks like 104 (not 106). And we should be noting every 5th enum anyway.

11:29 am

Sean Nash

Is this resolved now? Can we un-comment this code?

Is this resolved now? Can we un-comment this code?

11:19 am

wbracken  (deleted user)

Remove isPOSTComplete.

Remove isPOSTComplete.

8:40 am

Sean Nash

Can we remove "TODO" since implementation is not required at this time. Re-phrase comment to something like "If temp sensors require calibration, implement here."

Can we remove "TODO" since implementation is not required at this time. Re-phrase comment to something like "If temp sensors require calibration, implement here."

8:36 am

Sean Nash

POST does nothing? If always in progress, do we get stuck in this test?

POST does nothing? If always in progress, do we get stuck in this test?

8:31 am

Sean Nash

Remove blank line.

Remove blank line.

8:29 am

Sean Nash

Can these declarations be moved to top of function?

Can these declarations be moved to top of function?

8:25 am

Sean Nash

Lets get this question answered. I don't see the harm in having a max flow. I can't imagine we're ever seeing that much flow, so not sure why this has to be commented out while we sort out its exis...

Lets get this question answered. I don't see the harm in having a max flow. I can't imagine we're ever seeing that much flow, so not sure why this has to be commented out while we sort out its existence.

8:19 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

Monday 03 Oct 2022
5:07 pm

Darren Cox <darren_m@cox.net>

Merge branch 'DEN-14000-sw_dev_sprint_80_darren' into develop

Options
    • -5
    • +5
    ./AlarmMgmt.c
5:07 pm

Darren Cox <darren_m@cox.net>

DEN-14000 Modify Fault LED state to use fault active vs actual flashing LED state.

Options
    • -5
    • +5
    ./AlarmMgmt.c
  2. … 2 more files in changeset.
Friday 30 Sep 2022
3:29 pm

Darren Cox <darren_m@cox.net>

DEN-14000 merge to develop.

Options
    • -0
    • +2
    ./AlarmMgmtSWFaults.h
    • -0
    • +5
    ./SystemComm.c
    • -2
    • +32
    ./SystemCommMessages.c
    • -0
    • +3
    ./SystemCommMessages.h
  5. … 7 more files in changeset.
12:03 pm

Darren Cox <darren_m@cox.net>

DEN-14000 Removed CPLD interval override function. CPLD changes are broadcast on change event, not a timer.

Options
    • -4
    • +0
    ./SystemComm.c
    • -34
    • +0
    ./SystemCommMessages.c
    • -3
    • +0
    ./SystemCommMessages.h
  4. … 2 more files in changeset.
Thursday 29 Sep 2022
3:18 pm

Darren Cox <darren_m@cox.net>

DEN-14000 Added handling for CPLD broadcast interval override.

Options
    • -0
    • +4
    ./SystemComm.c
    • -0
    • +34
    ./SystemCommMessages.c
    • -0
    • +3
    ./SystemCommMessages.h
  4. … 1 more file in changeset.
Wednesday 28 Sep 2022
10:35 pm

wbracken  (deleted user)

Not really an output.

Not really an output.

8:55 pm

Dara Navaei

DEN-13834 addressed the code review comments and incorporated the PiSquare changes and addressed the reported VectorCAST issues

Options
    • -0
    • +1
    ./AlarmMgmtSWFaults.h
    • -0
    • +16
    ./FPGA.c
    • -3
    • +3
    ./Reservoirs.c
    • -0
    • +2
    ./SystemComm.c
  5. … 5 more files in changeset.
4:53 pm

Dara Navaei

Yes, do you see any logic issues in the statement?

Yes, do you see any logic issues in the statement?

4:48 pm

Dara Navaei

Done.

Done.

4:46 pm

Dara Navaei

Done.

Done.

4:43 pm

Dara Navaei

Done.

Done.