Vinayakam Mani

These set of valves defined as per system architecture (to make sure all valves initialized as expected).

These set of valves defined as per system architecture (to make sure all valves initialized as expected).

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

I would prefer to keep it here, as the start state (DRY_BICART_START_STATE) is always executed when the fill bicart flag (dryBiCartFillRequested) is not set. There are other execs, say fill chamber...

I would prefer to keep it here, as the start state (DRY_BICART_START_STATE) is always executed when the fill bicart flag (dryBiCartFillRequested) is not set. There are other execs, say fill chamber (execBicarbChamberFillMode) mode controls D80 in its sub states. So, not two execs controlling same valve. Once we identified where all modes using these execs, will come up with signals to avoid such scenarios.

Done.

Done.

Done

Done

Done

Done

Done.

Done.

RESOLVED IN CODEWALKTHROUGH

RESOLVED IN CODEWALKTHROUGH

error count or reading temperature?

error count or reading temperature?

Remove extra line.

Remove extra line.

Add the comments for coefficients. bring a top-level comment explaining the formula where these coefficients are being used.

Add the comments for coefficients. bring a top-level comment explaining the formula where these coefficients are being used.

Align the comments in fixed indentation throughout the file.

Align the comments in fixed indentation throughout the file.

space b/w '}' and 'Conductivity'. seen other places too in the file, please check and update.

space b/w '}' and 'Conductivity'. seen other places too in the file, please check and update.

UNINITIALIZED?

UNINITIALIZED?

Please add the comments mentioning the reason/purpose for the checking fluid level timeout alarm?

Please add the comments mentioning the reason/purpose for the checking fluid level timeout alarm?