common

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-5980 added the alarms to the alarm table and uncommented the load cells in DGDefs

Why commented out?

Why commented out?

Need to add these to end of alarm table below.

Need to add these to end of alarm table below.

Why commented out?

Why commented out?

Need to add these new alarms to alarm table below.

Need to add these new alarms to alarm table below.

Before switching to newly filled reservoir, we want to take a baseline weight (before starting UF with it). So we are initializing the starting and final weights for the reservoir here. As UF progr...

Before switching to newly filled reservoir, we want to take a baseline weight (before starting UF with it). So we are initializing the starting and final weights for the reservoir here. As UF progresses on this reservoir after we switch to it, the final weight will gradually increase and the delta between the two is how much UF we've taken using this reservoir.

DEN-5980 added more message IDs and alarms for NV operations

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Is that a rule in our coding standard?

Is that a rule in our coding standard?

Fixed.

Fixed.

Fixed.

Fixed.

Done.

Done.

It looks like the alarm stop signal flag is reset in another function? resetAlarmSignalFlags function.

It looks like the alarm stop signal flag is reset in another function? resetAlarmSignalFlags function.

Right after the if statement ends.

Right after the if statement ends.

Mismatch function name.

Mismatch function name.

Divide by 2 is averaging start and end rates of the ramp so that I can determine estimate of time needed to complete the ramp.

Divide by 2 is averaging start and end rates of the ramp so that I can determine estimate of time needed to complete the ramp.

Where?

Where?

Need to review doxygen style in this file.

Need to review doxygen style in this file.

All alarm user actions are blocked in this sub-mode. Alarm stop signal flag is covered in this reset function.

All alarm user actions are blocked in this sub-mode. Alarm stop signal flag is covered in this reset function.

Remove doxygen comment.

Remove doxygen comment.

Done.

Done.

Fixed.

Fixed.

Mismatch function name.

Mismatch function name.

Consider changing 60 to SEC_PER_MIN.

Consider changing 60 to SEC_PER_MIN.

We are taking both actions on the same reservoir. Sean Nash can provide more details.

We are taking both actions on the same reservoir. Sean Nash can provide more details.

DEN-5980 added more message IDs. Removed the older message IDs that were used for calibation

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Should not there be two IDs we are passing in? i.e., are we taking both actions on the same reservoir?

Should not there be two IDs we are passing in? i.e., are we taking both actions on the same reservoir?