astroufsc / chimera

Chimera - Observatory Automation System
http://chimera.sf.net/
GNU General Public License v2.0
36 stars 19 forks source link

Add new base instrument "Fan". #123

Closed tribeiro closed 8 years ago

tribeiro commented 9 years ago

Now, I understand this will probably create a lot of heated discussions around here (one more!). I initially though to include the domefan as an instance of dome. Problem is, you can easily have more then one dome fans installed and may want to control them independently. I could not though of a better way of doing it then to create a separate instrument (like filterwheel and camera).

wschoenell commented 9 years ago

I think that we could move to a more general instrument Fan which could be dome, telescope, etc...

wschoenell commented 9 years ago

Please add a configuration example to the fake instrument here: https://github.com/astroufsc/chimera/blob/37c8bd88f66f3f8e59e569f92ad09e6cdb8eeb9a/src/chimera/core/chimera.sample.config

Can you sort/group better the help options?

...
  Commands:
    --info              Print dome information
    --openFlap          Open dome flap
    --to=AZ             Move dome to AZ azimuth
    --setFanSpeed=SPEED
                        Set fan speed.
    --openSlit          Open dome slit
    --track             Track the telescope
    --startFan          Start dome fan
    --stopFan           Stop dome fan
    --closeFlap         Close dome flap
    --stand             Stop tracking the telescope (stand)
    --getFanSpeed       Print dome information
    --closeSlit         Close dome slit
...
tribeiro commented 9 years ago

Done

wschoenell commented 9 years ago

Still pending the help order...

tribeiro commented 9 years ago

Didn't saw that part...

Em ter, 10 de nov de 2015 às 14:52, William Schoenell < notifications@github.com> escreveu:

Still pending the help order...

— Reply to this email directly or view it on GitHub https://github.com/astroufsc/chimera/pull/123#issuecomment-155512840.

tribeiro commented 9 years ago

Ok, separated the actions on different groups.

wschoenell commented 8 years ago

I did some adaptions to this on my branch which I think that settle this PR.

  Dome lights control:
    --light-off         Turn light off
    --light-on          Turn light on
    --light-intensity=INTENSITY
                        Set light intensity to specified value.
    --lamp=LAMP         Calibration lamp to be used. [default=/Lamp/0]
...
  Dome Fans control:
    --fan-on            Start dome fan
    --fan-off           Stop dome fan
    --fan-speed=SPEED   Set fan speed.
    --fan=FAN           Fan instrument to be used. [default=/Fan/0]
  Telescope fan control:
    --fan-on            Start telescope fan
    --fan-off           Stop telescope fan
    --fan-speed=SPEED   Set fan speed.
    --fan=FAN           Fan instrument to be used. [default=/Fan/0]

One example of configuration for a dome and a telescope with fans/lamps:

telescope:
  name: fake
  type: FakeTelescope
  fans: ['/FakeFan/fakefan_tel']

dome:
  name: fake
  type: FakeDome
  mode: track
  telescope: /FakeTelescope/fake
  lamps: ['/FakeLamp/fake1']
  fans: ['/FakeFan/fake1', '/FakeFan/fake2']

lamps:
  - name: fake1
    type: FakeLamp
    dome_az: 0
    telescope_alt: 90
    telescope_az: 45

  - name: fake2
    type: FakeLamp
    dome_az: 10
    telescope_alt: 90
    telescope_az: 55

fans:
  - name: fake1
    type: FakeFan

  - name: fake2
    type: FakeFan

  - name: fakefan_tel
    type: FakeFan
wschoenell commented 8 years ago

Tested on T80 and on fake instrument. All working fine.