msr-consulting / exscalabar_server

Repository for the EXSCALABAR server.
http://www.msrconsults.com/ukmet-gh/exscalabar
0 stars 1 forks source link

Add Calibration Actor #38

Closed lo-co closed 7 years ago

lo-co commented 9 years ago

Actor should:

This actor will only be interruptable when requesting stop. Initially we will not allow editing of sequence on the fly. Maybe later?

lo-co commented 8 years ago

Added write to O3 LED. I will use this for all calibrations even though the title has O3 in it.

lo-co commented 8 years ago

Steps currently available in O3 sequence are:

Steps for which I have sketched out space but are not implemented are:

TODO

The Calbration Actor::Actor Core is implemented but totally untested.

lo-co commented 8 years ago

I am going to close this when the actor itself is in place. The steps should be addressed as individual issues as well as some of the other pieces. The actor should do these following things for closure:

lo-co commented 8 years ago

Looks like the Calibration Actor exits gracefully (i.e. all resources are released). I do need a stop calibration method (service method) that will return the system to it's original state.

lo-co commented 8 years ago

Upon exit, the Calibration Actor needs to:

On start:

lo-co commented 7 years ago

Removed the coercion in the O3LamFreq web service VI. Don't need this as we do it in the actual setting of the lamp frequency.

lo-co commented 7 years ago

Changed O3 Level STEP to O3 Lamp Frequency STEP. Removed conversion from step so now it is sending raw frequency.

lo-co commented 7 years ago

The actor works, but we need to make sure that all of the steps are executed properly.

JustinLangridge commented 7 years ago

Need to add control for pressure controller in the auto sequence setup options

lo-co commented 7 years ago

Added new step Pressure Control Rate STEP (bad name). This is triggered when the value P-Ctl is read from the file. On the UI side, I added a pressure control step in the calibration controller (ex.calibration.ctl.js).

JustinLangridge commented 7 years ago

Auto cals are working and thus closing this issue