Merge pull request #3 from pmbrull/feature/daily-stars

Daily stars instead of weekly
pmbrull · Apr 3, 2022 · 20df10f · 20df10f
2 parents d00e789 + 90abbb9
commit 20df10f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/openstats/components.py b/openstats/components.py
@@ -34,12 +34,12 @@ def stars_component(self):
         Prepare the graph to show the stars evolution
         and the differences
         """
-        df = self.data.stars_data().reset_index(level=0)
+        df = self.data.stars_data()
 
         if df is not None and not df.empty:
             current = int(df.iloc[-1].get("stars"))
-            last_week = int(df.iloc[-2].get("stars"))
-            last_month = int(df.iloc[-4].get("stars"))
+            last_week = int(df.iloc[-7].get("stars"))
+            last_month = int(df.iloc[-30].get("stars"))
 
         else:
             st.write("Error fetching Star data")

diff --git a/openstats/data.py b/openstats/data.py
@@ -59,7 +59,8 @@ def stars_data(self) -> Optional[DataFrame]:
             )
 
             df["date"] = pd.to_datetime(df["date"])
-            df = df.resample("W", on="date")[["stars"]].sum()
+            # This resample groups by week. Let's keep grouping by day.
+            # df = df.resample("W", on="date")[["stars"]].sum()
             df["stars"] = df["stars"].cumsum()
 
             return df