search

simonsobs / scheduler

Scheduler for SO
0 stars 1 forks source link

SATPolicy: add minimal elevation requirement #108

Closed guanyilun closed 1 month ago

guanyilun commented 1 month ago

addresses #107

mhasself commented 1 month ago

@guanyilun To fix sunsafe parking as per #111 , this min_el needs to be accessible within PlanMoves.apply. Can you please set up the infrastructure for that in whatever way makes most sense? Right now only very limited information makes it through, and a lot of that seems to be set at the per-platform Policy level rather than in the Policy base class.

guanyilun commented 1 month ago

Yes I agree the code organization is not the most sensible choice. I'm planning a code refactoring that gives build_op pipeline full access to policy configs.