Skip to content

Commit

Permalink
Remove m5 diff basis function; limit 40/70 alt
Browse files Browse the repository at this point in the history
  • Loading branch information
rhiannonlynne committed Dec 11, 2024
1 parent 17ec75c commit fa69b3a
Showing 1 changed file with 9 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
# along with this program. If not, see <https://www.gnu.org/licenses/>.

import numpy as np
from rubin_scheduler.scheduler import detailers, example, features
from rubin_scheduler.scheduler import basis_functions, detailers, example, features
from rubin_scheduler.scheduler.schedulers import CoreScheduler
from rubin_scheduler.scheduler.surveys import BlobSurvey
from rubin_scheduler.scheduler.utils import CurrentAreaMap, Footprint
Expand Down Expand Up @@ -53,8 +53,8 @@ def get_scheduler():
# Erik says to make wind speed minor so set limit high
wind_speed_maximum=50.0,
# I think these are the values appropriate for alt limits now?
min_alt=30,
max_alt=80,
min_alt=40,
max_alt=70,
min_az=0,
max_az=360,
# Avoid going into avoidance regions 30 minutes into the future
Expand Down Expand Up @@ -119,6 +119,12 @@ def get_scheduler():
reward_functions = [(i[0], i[1]) for i in rf1 if i[1] > 0] + [
(i[0], i[1]) for i in rf2 if i[1] > 0
]
# Remove the M5Diff basis functions as unusable at present
reward_functions = [
(bf, weight)
for (bf, weight) in reward_functions
if not isinstance(bf, basis_functions.M5DiffBasisFunction)
]

# unpack the basis functions and weights
reward_basis_functions_weights = [val[1] for val in reward_functions]
Expand Down

0 comments on commit fa69b3a

Please sign in to comment.