HD-DEN-6200

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Before, the HD sends target temperature using another message. So now we add the target temperature to this command, so DG can handle the full command.

Before, the HD sends target temperature using another message. So now we add the target temperature to this command, so DG can handle the full command.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

What made us change our mind and pass in set temperature command compared to before (just turning OFF and ON)?

What made us change our mind and pass in set temperature command compared to before (just turning OFF and ON)?

Added the switch statement. Added comment and TODO.

Added the switch statement.
Added comment and TODO.

Consider having a switch statement here for each reject reason code. Maybe we're only handling the invalid parameter reason for now, but we may change our minds later. You can group the rest of the...

Consider having a switch statement here for each reject reason code. Maybe we're only handling the invalid parameter reason for now, but we may change our minds later. You can group the rest of the reasons+default with just a comment that our state machines will detect and handle failure of DG to advance for now. This will make it clear that we didn't forget about the other cases.

Signals have been forwarded to appropriate sub-mode to handle.

Signals have been forwarded to appropriate sub-mode to handle.

These states rely on valves setting change on the transition of the previous state. Therefore, we need to start from the beginning state.

These states rely on valves setting change on the transition of the previous state. Therefore, we need to start from the beginning state.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Why are there several cases with same outcome in this function?

Why are there several cases with same outcome in this function?

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.

Added handler to DG command response.

Added handler to DG command response.

Pretreatment mode does not have data needed to be broadcast. We might have more data once the mode is more mature.

Pretreatment mode does not have data needed to be broadcast.
We might have more data once the mode is more mature.

Fixed. This signal indicates the start of treatment.

Fixed. This signal indicates the start of treatment.

Added clarification to comment and name.

Added clarification to comment and name.

The message has a command id field.

The message has a command id field.

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.