Skip to content

Commit

Permalink
Merge pull request #40 from lsst/tickets/PREOPS-4105
Browse files Browse the repository at this point in the history
Tickets/PREOPS-4105: Change how survey number is queried in subtitle.
  • Loading branch information
alserene authored Oct 20, 2023
2 parents b393838 + 265eb14 commit 278e7e8
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
7 changes: 3 additions & 4 deletions schedview/app/scheduler_dashboard/scheduler_dashboard.py
Original file line number Diff line number Diff line change
Expand Up @@ -1028,13 +1028,12 @@ def generate_dashboard_subtitle(self):
if not self._display_dashboard_data:
return ""
maps = ["u_sky", "g_sky", "r_sky", "i_sky", "z_sky", "y_sky", "reward"]
survey = self._survey_name.split(":")[0]
if not self._display_reward and self.survey_map in maps:
return f"\nTier {self._tier[-1]} - Survey {survey} - Map {self._map_name}"
return f"\nTier {self._tier[-1]} - Survey {self._survey} - Map {self._map_name}"
elif not self._display_reward and self.survey_map not in maps:
return f"\nTier {self._tier[-1]} - Survey {survey} - Reward {self._map_name}"
return f"\nTier {self._tier[-1]} - Survey {self._survey} - Reward {self._map_name}"
else:
return f"\nTier {self._tier[-1]} - Survey {survey} - Reward {self._reward_name}"
return f"\nTier {self._tier[-1]} - Survey {self._survey} - Reward {self._reward_name}"

def generate_summary_table_heading(self):
"""Select the summary table heading based on whether data is being
Expand Down
4 changes: 3 additions & 1 deletion tests/test_scheduler_dashboard.py
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,9 @@ def test_read_scheduler(self):
def test_title(self):
self.scheduler._tier = "tier 2"
self.scheduler._survey = 0
self.scheduler._survey_name = "0: survey"
self.scheduler.param["survey_map"].objects = ["reward"]
self.scheduler.survey_map = "reward"
self.scheduler._map_name = "reward"
self.scheduler._display_dashboard_data = True
title = self.scheduler.generate_dashboard_subtitle()
tier = self.scheduler._tier[-1]
Expand Down

0 comments on commit 278e7e8

Please sign in to comment.