dgfirmware / firmware / App / Controllers

Heaters.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 10
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 seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Sunday 09 Oct 2022
8:25 pm

Dara Navaei

DEN-13834 added the drain pump current and direction alarms. Updated the heaters driver disinfect control. Added a check to ignore the trimmer heater stop command from HD if DG is in the heat disinfect or chemical disinfect

Options
    • -33
    • +29
    ./Heaters.c
  2. … 6 more files in changeset.
Saturday 08 Oct 2022
2:17 pm

Dara Navaei

DEN-13834 added more self test alarms. Added FPGA functions to get the drain pump current feedbakc, speed feedback, and direction

Options
    • -2
    • +2
    ./Heaters.c
  2. … 5 more files in changeset.
Thursday 06 Oct 2022
11:18 am

wbracken  (deleted user)

Alarm for invalid loadCellID?

Alarm for invalid loadCellID?

11:16 am

wbracken  (deleted user)

Should there be an alarm if loadCellID is invalid?

Should there be an alarm if loadCellID is invalid?

11:13 am

wbracken  (deleted user)

Why the change?

Why the change?

10:02 am

Dara Navaei

Merged DEN-13834

Options
    • -11
    • +21
    ./Heaters.c
  2. … 17 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
3:45 pm

Dara Navaei

DEN-13834 updated the code

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

Darren Cox <darren_m@cox.net>

DEN-14000 merge to develop.

Options
    • -4
    • +4
    ./Heaters.c
  2. … 10 more files 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
    • -6
    • +6
    ./Heaters.c
  2. … 8 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.