Fix chart to fetch all data for timeframe, not limited

Author: buzzkillb

chart_service.py

@@ -215,10 +215,7 @@ async def get_chart_bytes(
     ) -> Optional[bytes]:
         """Get price history from DB and generate chart."""
         max_points = 800 if hours <= 24 else 1200 if hours <= 168 else 1500
-        limit = max_points * 2
-        history = await db.get_price_history(
-            crypto, hours=hours, limit=limit, descending=True
-        )
+        history = await db.get_price_history(crypto, hours=hours, descending=True)
 
         if not history or len(history) < 2:
             return None

database.py

@@ -253,16 +253,27 @@ async def get_price_history(
         query = query.replace("ORDER BY timestamp ASC", f"ORDER BY timestamp {order}")
 
         async with self.pool.acquire() as conn:
-            rows = await conn.fetch(query, crypto_name.upper(), cutoff, limit)
+            if limit is None:
+                rows = await conn.fetch(query, crypto_name.upper(), cutoff)
+            else:
+                rows = await conn.fetch(query, crypto_name.upper(), cutoff, limit)
 
             if not rows and bucket_type != "raw":
-                query = f"""
-                    SELECT timestamp, price FROM prices
-                    WHERE crypto_name = $1 AND timestamp > $2
-                    ORDER BY timestamp {order}
-                    LIMIT $3
-                """
-                rows = await conn.fetch(query, crypto_name.upper(), cutoff, limit)
+                if limit is None:
+                    query = f"""
+                        SELECT timestamp, price FROM prices
+                        WHERE crypto_name = $1 AND timestamp > $2
+                        ORDER BY timestamp {order}
+                    """
+                    rows = await conn.fetch(query, crypto_name.upper(), cutoff)
+                else:
+                    query = f"""
+                        SELECT timestamp, price FROM prices
+                        WHERE crypto_name = $1 AND timestamp > $2
+                        ORDER BY timestamp {order}
+                        LIMIT $3
+                    """
+                    rows = await conn.fetch(query, crypto_name.upper(), cutoff, limit)
 
             return [(r["timestamp"], float(r["price"])) for r in rows]