Modes

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Question was in relation to DG gen idle sub-state, not HD mode.

Question was in relation to DG gen idle sub-state, not HD mode.

In pre-treat, there were transfers from one reservoir to the other that caused a false alarm. Figuring that out in SW at this level seemed overly complex. Also, the ticket notes this problem in Tre...

In pre-treat, there were transfers from one reservoir to the other that caused a false alarm. Figuring that out in SW at this level seemed overly complex. Also, the ticket notes this problem in Treatment only.

Why is this reservoir check only happening in this state? Recommend creating a separate function to perform the check and call that function from gen idle exec function (i.e. do check in all gen id...

Why is this reservoir check only happening in this state? Recommend creating a separate function to perform the check and call that function from gen idle exec function (i.e. do check in all gen idle states).

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

Updated.

Updated.

DG idle reservoir out of range - rewording " DG idle reservoir weight out of range" would be better?

DG idle reservoir out of range - rewording " DG idle reservoir weight out of range" would be better?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DG-DEN-16666_Log Passing Results OF Self Test TO UI Log
DG-DEN-16666_Log Passing Results OF Self Test TO UI Log
Updated.

Updated.