dgfirmware / firmware / App

DGCommon.h

Clone Tools

Stats

Commits this week: 0
Total Committers: 8
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
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?

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.

Wednesday 28 Sep 2022
10:35 pm

wbracken  (deleted user)

Not really an output.

Not really an output.

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.

4:38 pm

Dara Navaei

This implementation is not required right now. We are not calibrating temperature sensors.

This implementation is not required right now. We are not calibrating temperature sensors.

4:37 pm

Dara Navaei

That is 110.

That is 110.

4:36 pm

Dara Navaei

This was because I merged staging into my branch. I has been removed again.

This was because I merged staging into my branch. I has been removed again.

4:35 pm

Dara Navaei

Done.

Done.