Controllers

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Done

Done

Done

Done

You at least need to add a comment explaining what they are.

You at least need to add a comment explaining what they are.

Done

Done

No, these are the minimum values to trigger the alarm. But they are named maximums.

No, these are the minimum values to trigger the alarm. But they are named maximums.

Yes these are for software configuration disable only so I did not make any #defines for them.

Yes these are for software configuration disable only so I did not make any #defines for them.

Yes, so name should reflect that. These names suggest they are minimum pressures below which we would alarm.

Yes, so name should reflect that. These names suggest they are minimum pressures below which we would alarm.

Align comment.

Align comment.

Magic numbers?

Magic numbers?

This is the minimum values that are needed to clear the alarm.

This is the minimum values that are needed to clear the alarm.

This is the minimum values that are needed to clear the alarm.

This is the minimum values that are needed to clear the alarm.

These 2 values appear to be minimum recovery thresholds.

These 2 values appear to be minimum recovery thresholds.

These to values appear to be minimums. Why are they being called maximums?

These to values appear to be minimums. Why are they being called maximums?

DG-DEN-15973_Alarm 48 UF Rate Tare Error Triggered During Treatment
DG-DEN-15973_Alarm 48 UF Rate Tare Error Triggered During Treatment
RESOLVED in CODE WALTHROUGH

RESOLVED in CODE WALTHROUGH

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Corrected

Corrected

Keep declarations at top of scope. Just have assignments down here.

Keep declarations at top of scope. Just have assignments down here.

Removed blank line

Removed blank line

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done.

Done.