dgfirmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-11928 Updated per peer review comments

    • -14
    • +27
    /firmware/App/Modes/ModeGenIdle.c
Added signal function setBadAvgConductivityDetectedFlag( BOOL flag ) in ModeGenIdle.

Added signal function setBadAvgConductivityDetectedFlag( BOOL flag ) in ModeGenIdle.

Initialized badFillState where it is declared in the private data section.

Initialized badFillState where it is declared in the private data section.

Added genIdleState = DG_GEN_IDLE_MODE_STATE_START;

Added
genIdleState = DG_GEN_IDLE_MODE_STATE_START;

Removed line //requestNewOperationMode( DG_MODE_GENE );

Removed line

//requestNewOperationMode( DG_MODE_GENE );

Removed line //activateAlarmNoData( ALARM_ID_DG_ACID_BOTTLE_LOW_VOLUME );

Removed line

//activateAlarmNoData( ALARM_ID_DG_ACID_BOTTLE_LOW_VOLUME );

Moved check for empty bottles before if ( TRUE == isReservoirTarePending() )

Moved check for empty bottles before if ( TRUE == isReservoirTarePending() )

Removed else requestNewOperationMode( DG_MODE_GENE );

Removed else requestNewOperationMode( DG_MODE_GENE );

Uncommented requestNewOperationMode( )

Uncommented requestNewOperationMode( )

Added doxygen comment DG_HANDLE_BAD_FILL_STATE_START = 0, ///< Bad fill start state

Added doxygen comment

DG_HANDLE_BAD_FILL_STATE_START = 0, ///< Bad fill start state

Added doxygen comment ALARM_ID_INLET_WATER_HIGH_CONDUCTIVITY = 106, ///< DG inlet water conductivity is greater than threshold

Added doxygen comment

ALARM_ID_INLET_WATER_HIGH_CONDUCTIVITY = 106, ///< DG inlet water conductivity is greater than threshold

Removed U32 fillToVolumeML = 1000; Added #define BAD_FLUSH_FILL_TARGET_VOLUME_ML 1000

Removed U32 fillToVolumeML = 1000;
Added #define BAD_FLUSH_FILL_TARGET_VOLUME_ML 1000

Removed 2 unused thresholds: #define CONDUCTIVITY_WHEN_ACID_JUG_EMPTY 10000.0 #define CONDUCTIVITY_WHEN_BICARB_JUG_EMPTY 12000.0

Removed 2 unused thresholds:

#define CONDUCTIVITY_WHEN_ACID_JUG_EMPTY 10000.0
#define CONDUCTIVITY_WHEN_BICARB_JUG_EMPTY 12000.0

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Maybe remove thresholds for now as we are not using them yet.

Maybe remove thresholds for now as we are not using them yet.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Don't need this else and request for idle.

Don't need this else and request for idle.