Fixed Issue #239: Off by one day in charts and csv files

Author: Andrew DiBiasio

src/penn_chime/models.py

@@ -296,7 +296,7 @@ def build_dispositions_df(
 
 def build_admits_df(dispositions_df: pd.DataFrame) -> pd.DataFrame:
     """Build admits dataframe from dispositions."""
-    admits_df = dispositions_df.iloc[:-1, :] - dispositions_df.shift(1)
+    admits_df = dispositions_df.iloc[:, :] - dispositions_df.shift(1)
     admits_df.day = dispositions_df.day
     admits_df.date = dispositions_df.date
     return admits_df

src/penn_chime/presentation.py

@@ -327,7 +327,7 @@ def display_sidebar(st, d: Parameters) -> Parameters:
             docs_url=DOCS_URL
         )
     )
-    n_days = n_days_input() + 1 # Overcome zero index error with panda df
+    n_days = n_days_input()
     max_y_axis_set = max_y_axis_set_input()
 
     max_y_axis = None
@@ -472,7 +472,7 @@ def write_footer(st):
 
 
 def display_download_link(st, filename: str, df: pd.DataFrame):
-    csv = dataframe_to_base64(df[:-1]) # Overcome zero index error with panda df
+    csv = dataframe_to_base64(df)
     st.markdown(
         """
         <a download="{filename}" href="data:file/csv;base64,{csv}">Download {filename}</a>