dgfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Done

Done

Done

Done

Done

Done

Done. Added a check in the function in the Mode Fill like the rest of the modes and drivers.

Done. Added a check in the function in the Mode Fill like the rest of the modes and drivers.

Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Do this (request fill mode) in the testSetModeFillForCal() function above (and only if Dialin is logged in).

Do this (request fill mode) in the testSetModeFillForCal() function above (and only if Dialin is logged in).

Make sure Dialin is logged in before accepting command.

Make sure Dialin is logged in before accepting command.

If this is a Dialin command, make sure Dialin is logged in before acting on it (throughout this module).

If this is a Dialin command, make sure Dialin is logged in before acting on it (throughout this module).

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

If these are Dialin commands, add a condition around this switch that Dialin has logged in.

If these are Dialin commands, add a condition around this switch that Dialin has logged in.

These are Dialin commands? If so, prefix each enum with CMD_ and clarify this in comment.

These are Dialin commands? If so, prefix each enum with CMD_ and clarify this in comment.

CD1 is used to check the difference between CD1 and CD2. CD1 is not used to check the reading of acid.

CD1 is used to check the difference between CD1 and CD2. CD1 is not used to check the reading of acid.

Will address in DEN-16693.

Will address in DEN-16693.

All of these for loops are not following our coding standards. Should have bracketed scope even if single line of code in loop.

All of these for loops are not following our coding standards. Should have bracketed scope even if single line of code in loop.

Does this mean we aren't checking the CD1 sensor at all?

Does this mean we aren't checking the CD1 sensor at all?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Yes this is still used in the deliver state. I changed the names in the #defines

Yes this is still used in the deliver state. I changed the names in the #defines

Is this 5% definition still used? I don't think these are technically percentages (i.e. we do not mean 0.2%, we mean 20%).

Is this 5% definition still used?
I don't think these are technically percentages (i.e. we do not mean 0.2%, we mean 20%).

DG-DEN-16666_Log Passing Results OF Self Test 2
DG-DEN-16666_Log Passing Results OF Self Test 2
DG-DEN-16674_Td2 Calibration Feature
DG-DEN-16674_Td2 Calibration Feature