simonsobs / scheduler

Scheduler for SO
0 stars 1 forks source link

Azimuth inconsistency between acu.move_to() and tag #69

Closed yoshinori-0778 closed 5 months ago

yoshinori-0778 commented 5 months ago

We found generated schedule has Azimuth inconsistency between acu.move_to(az=-89.108, el=60.0) and tag='271-311'. We noticed when we faced "ProgramTrack_position_failure" because ACU limits for the SATP are -90 to 450 degrees.

run.acu.move_to(az=-89.108, el=60.0)
run.wait_until('2024-06-02T01:28:12+00:00')
# hwp disabled
scan_stop = datetime.datetime(2024, 6, 2, 2, 28, tzinfo=datetime.timezone.utc)
if datetime.datetime.now(tz=UTC) < scan_stop - datetime.timedelta(minutes=10):
    run.seq.scan(
        description='NW. First choice',
        stop_time='2024-06-02T02:28:00+00:00',
        width=40.0, az_drift=0,
        subtype='cmb', tag='271-311',
    )
    run.smurf.bias_step(concurrent=True)

Used scripts:

Used version:

ykyohei commented 5 months ago

I guess we can solve this by changing az_limits? Recently satp1 policy changed this value from [-90, 450] to [-45, 405] https://github.com/simonsobs/scheduler/blob/026bfc77498c598b8c02e15cc314b15a29332336/src/schedlib/policies/satp1.py#L234

ykyohei commented 5 months ago

I think we don't have this issue after changing az_limits