Comment with link to #381

CodeForPhilly · Mar 30, 2020 · 3c65eef · 3c65eef
1 parent cc0360c
commit 3c65eef
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/penn_chime/models.py b/src/penn_chime/models.py
@@ -182,6 +182,8 @@ def get_loss(self) -> float:
 
 
 def get_argmin_ds(census_df: pd.DataFrame, current_hospitalized: float) -> float:
+    # By design, this forbids choosing a day after the peak
+    # If that's a problem, see #381
     peak_day = census_df.hospitalized.argmax()
     losses_df = (census_df.hospitalized[:peak_day] - current_hospitalized) ** 2.0
     return losses_df.argmin()