AlarmDefs.h

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
I think p comes before u. Move up one.

I think p comes before u. Move up one.

We should initialize BP module from main at startup so we can use it in any mode.

We should initialize BP module from main at startup so we can use it in any mode.

I think this should be executed from TaskGeneral so we can do BP things in any mode.

I think this should be executed from TaskGeneral so we can do BP things in any mode.

What is "e"? I think this might be a typo and you should put it back the way it was.

What is "e"? I think this might be a typo and you should put it back the way it was.

please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

Alarm id should be 196? Description looks same as 194.

Alarm id should be 196?
Description looks same as 194.

fixed thanks

fixed thanks

Should we change msg ID from speed to rate?

Should we change msg ID from speed to rate?

fixed thanks

fixed thanks

its a speed in rpm. updated function to convert target rate in ml/min to target speed in rpm and use that for the difference.

its a speed in rpm. updated function to convert target rate in ml/min to target speed in rpm and use that for the difference.

fixed thanks

fixed thanks

this is a set speed in rpm

this is a set speed in rpm

this is speed in rpm

this is speed in rpm

fixed thanks

fixed thanks

fixed thanks

fixed thanks

fixed thanks

fixed thanks

Change speed to rate.

Change speed to rate.

pumpTargetRate

pumpTargetRate

Change function name to say rate instead of speed

Change function name to say rate instead of speed

Is this a rate too?

Is this a rate too?

Is this a rate too?

Is this a rate too?

Change speed to rate.

Change speed to rate.

Is currentPumpSpeed actually a rate?

Is currentPumpSpeed actually a rate?

Is this comment correct? Seems like we're stopping pump if requested to turn off.

Is this comment correct? Seems like we're stopping pump if requested to turn off.

Is currentPumpSpeed also a rate? If not, we're subtracting a speed from a rate which doesn't make much sense.

Is currentPumpSpeed also a rate? If not, we're subtracting a speed from a rate which doesn't make much sense.

Align comment.

Align comment.

Align comment. Change "speed" to "rate in comment too. Also, I don't see you initializing these rates.

Align comment. Change "speed" to "rate in comment too.
Also, I don't see you initializing these rates.