RO-LDT-566

Align comment.

Align comment.

Where did these move to?

Where did these move to?

These broadcast changes look like temporary test changes. Restore before merging this branch to staging.

These broadcast changes look like temporary test changes. Restore before merging this branch to staging.

This "if" is not necessary. Just assign to TRUE regardless.

This "if" is not necessary. Just assign to TRUE regardless.

Add comment to right. And it looks like this flag only relates to flow control - so variable name and comment should reflect that.

Add comment to right.
And it looks like this flag only relates to flow control - so variable name and comment should reflect that.

Align "=".

Align "=".

Should this be an "else if"?

Should this be an "else if"?

Add blank line after declarations.

Add blank line after declarations.

Remove extra blank line.

Remove extra blank line.

Are we going to change back to duty cycle % as control signal output?

Are we going to change back to duty cycle % as control signal output?

Realign comments.

Realign comments.

No and yes. This is for debugging purposes since Dialin will keep a history of events that have happened rather than the alarm module that will only contain the last received alarm trigger. I only ...

No and yes. This is for debugging purposes since Dialin will keep a history of events that have happened rather than the alarm module that will only contain the last received alarm trigger. I only want to keep this until we start enforcing alarms and transitioning into fault.

Is this adding anything that the trigger alarm message doesn't already do?

Is this adding anything that the trigger alarm message doesn't already do?

RO-LDT-566_Water Treatment And Degassing FP 2
RO-LDT-566_Water Treatment And Degassing FP 2
When commenting out problematic alarms (temporarily), can we at least send an event w/ same 2 datas for logging purposes so we can investigate later?

When commenting out problematic alarms (temporarily), can we at least send an event w/ same 2 datas for logging purposes so we can investigate later?

RO-LDT-566_Water Treatment And Degassing FP 2
RO-LDT-566_Water Treatment And Degassing FP 2
RO-LDT-566_Water Treatment And Degassing FP 2
RO-LDT-566_Water Treatment And Degassing FP 2
Added TODO. It'll be implemented in the next branch regarding stop/pauses states and alarm requirements.

Added TODO. It'll be implemented in the next branch regarding stop/pauses states and alarm requirements.

Remove blank line.

Remove blank line.

I think FP and DD need to handle alarm trigger, clear, and clear condition messages (msg ID 2,3,4) from other s/w stacks.

I think FP and DD need to handle alarm trigger, clear, and clear condition messages (msg ID 2,3,4) from other s/w stacks.

I would put normal messages above Dialin messages in this list - basically ordering this list in msg enum order.

I would put normal messages above Dialin messages in this list - basically ordering this list in msg enum order.

Remove blank line.

Remove blank line.