From c49634b0ed46c18d1d280ad539de2bd32f9d26a4 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Mon, 13 Apr 2026 13:48:39 +0000
Subject: [PATCH 1/2] feat(ux): Add thousands separators to financial outputs

- Add thousands separators to print statements in `bitcoin_trading_simulation.py` to make the daily trading ledger more readable.

Co-authored-by: EiJackGH <172181576+EiJackGH@users.noreply.github.com>
---
 bitcoin_trading_simulation.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/bitcoin_trading_simulation.py b/bitcoin_trading_simulation.py
index 0d46e73..7e9b423 100644
--- a/bitcoin_trading_simulation.py
+++ b/bitcoin_trading_simulation.py
@@ -102,7 +102,7 @@ def simulate_trading(signals, initial_cash=10000, quiet=False):
             portfolio.loc[i, 'btc'] += btc_to_buy
             portfolio.loc[i, 'cash'] -= btc_to_buy * row['price']
             if not quiet:
-                print(f"{Colors.GREEN}🟢 Day {i}: Buy {btc_to_buy:.4f} BTC at ${row['price']:.2f}{Colors.ENDC}")
+                print(f"{Colors.GREEN}🟢 Day {i}: Buy {btc_to_buy:,.4f} BTC at ${row['price']:,.2f}{Colors.ENDC}")
 
         # Sell signal
         elif row['positions'] == -2.0:
@@ -110,14 +110,14 @@ def simulate_trading(signals, initial_cash=10000, quiet=False):
                 cash_received = portfolio.loc[i, 'btc'] * row['price']
                 portfolio.loc[i, 'cash'] += cash_received
                 if not quiet:
-                    print(f"{Colors.FAIL}🔴 Day {i}: Sell {portfolio.loc[i, 'btc']:.4f} BTC at ${row['price']:.2f}{Colors.ENDC}")
+                    print(f"{Colors.FAIL}🔴 Day {i}: Sell {portfolio.loc[i, 'btc']:,.4f} BTC at ${row['price']:,.2f}{Colors.ENDC}")
                 portfolio.loc[i, 'btc'] = 0
 
         portfolio.loc[i, 'total_value'] = portfolio.loc[i, 'cash'] + portfolio.loc[i, 'btc'] * row['price']
 
         if not quiet:
-            print(f"Day {i}: Portfolio Value: ${portfolio.loc[i, 'total_value']:.2f}, "
-                  f"Cash: ${portfolio.loc[i, 'cash']:.2f}, BTC: {portfolio.loc[i, 'btc']:.4f}")
+            print(f"Day {i}: Portfolio Value: ${portfolio.loc[i, 'total_value']:,.2f}, "
+                  f"Cash: ${portfolio.loc[i, 'cash']:,.2f}, BTC: {portfolio.loc[i, 'btc']:,.4f}")
 
     if quiet and sys.stdout.isatty():
         print()

From 1afeb7747c2a8c847def39e87baa688782e97578 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Mon, 13 Apr 2026 13:54:27 +0000
Subject: [PATCH 2/2] fix(ci): Remove `venv` from requirements to fix CI
 failure

- Removed `venv` from `requirements.txt` because it is a standard library module and caused `pip install` to fail.
- Note: This commit is on the same branch that already includes the UX fix for adding thousands separators.

Co-authored-by: EiJackGH <172181576+EiJackGH@users.noreply.github.com>
---
 requirements.txt | 1 -
 1 file changed, 1 deletion(-)

diff --git a/requirements.txt b/requirements.txt
index cfaa995..4ad1501 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -1,4 +1,3 @@
 numpy
 pandas
 requests
-venv