Modes

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-14547: Addressing code review comments.

transitionToNewOperationMode() is currently void. This would have to change to returning the submode enum. This was an issue found in testing the events that we miss the event transition from going...

transitionToNewOperationMode() is currently void. This would have to change to returning the submode enum. This was an issue found in testing the events that we miss the event transition from going to new modes and needed the code change down at line ~380.

Do we want to change transitionToNewOperationMode() to return sub mode?

Fixed merge conflict.

  1. … 6 more files in changeset.
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

DEN-14749 fixed the exit status of the Nelson labs heat disinfect positive control mode

DG-DEN-14749_Syringe Pump Plunger Remains Extended During Pretreatment Causing HD Software Fault
DG-DEN-14749_Syringe Pump Plunger Remains Extended During Pretreatment Causing HD Software Fault
Fixed merge conflicts.

  1. … 6 more files in changeset.
Add currentMode, currentSubMode to inputs and outputs

Add currentMode, currentSubMode to inputs and outputs

currentSubMode is an input now.

currentSubMode is an input now.

Restore this blank line.

Restore this blank line.

Should currentSubMode be assigned to return value here?

Should currentSubMode be assigned to return value here?

Move blank line from before to after declaration.

Move blank line from before to after declaration.

Remove blank lines.

Remove blank lines.

DG-DEN-14763_DG HD Dev HD DG Dvt Update Part 10
DG-DEN-14763_DG HD Dev HD DG Dvt Update Part 10
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 9 more files in changeset.
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

DEN-14604 added VectorCAST from PiSquare and addressed code review comments

  1. … 1 more file in changeset.
RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

Done

Done

The enum comments are now non-doxygen style, but the enum is still in the group. Move it below the /*@}/ to get it out of HeatersState group.

The enum comments are now non-doxygen style, but the enum is still in the group. Move it below the /*@}/ to get it out of HeatersState group.

Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Shouldn't this one stay "MIN"?

Shouldn't this one stay "MIN"?