Skip to content

🎨 Palette: Enhance CLI Report with ASCII Dashboard & Stats#62

Merged
EiJackGH merged 2 commits intomainfrom
palette-enhanced-cli-report-10784789726884231400
Feb 20, 2026
Merged

🎨 Palette: Enhance CLI Report with ASCII Dashboard & Stats#62
EiJackGH merged 2 commits intomainfrom
palette-enhanced-cli-report-10784789726884231400

Conversation

@EiJackGH
Copy link
Owner

💡 What:

  • Removed duplicate Colors class definition in bitcoin_trading_simulation.py.
  • Fixed a bug where "Daily Trading Ledger" was printed twice.
  • Enhanced the "Final Portfolio Performance" report with a structured ASCII box layout.
  • Added new statistics to the report: ROI (%), Total Trades (Buys/Sells), and comparison vs Buy & Hold.

🎯 Why:

  • The original output was messy (duplicate headers) and hard to read (plain text list).
  • Users need key metrics like ROI and trade activity at a glance.
  • The new layout reduces cognitive load and makes the tool feel more professional.

📸 Before:

------ Daily Trading Ledger ------

------ Daily Trading Ledger ------
...
------ Final Portfolio Performance ------
Initial Cash: $10000.00
Final Portfolio Value: $9896.74
📉 Profit/Loss: $-103.26
Buy and Hold Strategy Value: $10437.41
-----------------------------------------

📸 After:

╔════════════════════════════════════════════╗
║       Final Portfolio Performance          ║
╠════════════════════════════════════════════╣
║ Initial Cash:                   $10,000.00 ║
║ Final Portfolio Value:           $9,775.97 ║
║ Profit/Loss:                      -$224.03 ║
║ ROI:                                -2.24% ║
╠════════════════════════════════════════════╣
║ Total Trades:                            2 ║
║   - Buys:                                1 ║
║   - Sells:                               1 ║
╠════════════════════════════════════════════╣
║ Buy & Hold Value:               $10,541.57 ║
║ vs Buy & Hold:                    -$765.59 ║
╚════════════════════════════════════════════╝

Accessibility:

  • Maintained high contrast colors (Green/Red) but ensured layout is readable without them.
  • Used clear labels and alignment.
  • Emojis (hardcoded) provide visual cues even in no-color mode.

PR created automatically by Jules for task 10784789726884231400 started by @EiJackGH

…and redundant header printing. Enhance the final simulation report with an ASCII box layout, adding ROI, trade counts, and strategy comparison for better UX. Update tests to reflect changes.

Co-authored-by: EiJackGH <172181576+EiJackGH@users.noreply.github.com>
@google-labs-jules
Copy link
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

…dency, and enhance CLI UX with ASCII dashboard.

- Restore `bitcoin.py` with `get_bitcoin_price` and `calculate_value` functions to resolve `ModuleNotFoundError`.
- Add `requests` to `requirements.txt`.
- Fix Flake8 linting errors in `bitcoin_trading_simulation.py`, `test_bitcoin.py`, and `test_simulation.py` (whitespace, blank lines, imports).
- Enhance `bitcoin_trading_simulation.py` final report with an ASCII box layout, ROI, and trade stats.
- Consolidate `Colors` class in `bitcoin_trading_simulation.py` and remove duplicate `argparse` import.

Co-authored-by: EiJackGH <172181576+EiJackGH@users.noreply.github.com>
@EiJackGH EiJackGH merged commit dadbe2a into main Feb 20, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant