Controllers

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

Done.

The other states ignore this flag and I have to set it here since this is a request.

The other states ignore this flag and I have to set it here since this is a request.

For safety I do it here.

For safety I do it here.

Done

Done

Done

Done

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Removed the release check.

Removed the release check.

Done

Done

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Why isn't this command allowed for release version?

Why isn't this command allowed for release version?

Is this zero temporary? If so, add TODO.

Is this zero temporary? If so, add TODO.

I don't think we should exit normal mode in TreatmentStop sub-mode.

I don't think we should exit normal mode in TreatmentStop sub-mode.

Should we exit blood leak normal mode in here somewhere?

Should we exit blood leak normal mode in here somewhere?

Ask Behrouz if UI is still looking for this. Remove if not.

Ask Behrouz if UI is still looking for this. Remove if not.

Remove extra blank line. And why is there any blank line here?

Remove extra blank line. And why is there any blank line here?

Seems like we're setting a start time offset here, not a time stamp.

Seems like we're setting a start time offset here, not a time stamp.

Should be rpmAlarmStartTime

Should be rpmAlarmStartTime

Did we want this offset to revert to zero here or should Dialin have to do a reset?

Did we want this offset to revert to zero here or should Dialin have to do a reset?

I don't think this results in an elapsed time. It's just a different start time. The calcTimeSince below gives you the elapsed time.

I don't think this results in an elapsed time. It's just a different start time. The calcTimeSince below gives you the elapsed time.

Should we set this flag only if we are in normal state?

Should we set this flag only if we are in normal state?

Remove extra blank line.

Remove extra blank line.

Should this be set to FALSE?

Should this be set to FALSE?

"reset" should be "rest" I think.

"reset" should be "rest" I think.

Agree. Dara, make these look something like "C_CALIBRATION_CMD".

Agree. Dara, make these look something like "C_CALIBRATION_CMD".

We will want to tighten these eventually.

We will want to tighten these eventually.