🎨 Palette: Add tooltips to technical inputs and fix sidebar crash#47
🎨 Palette: Add tooltips to technical inputs and fix sidebar crash#47
Conversation
- Added `help` text to sidebar inputs for better usability - Added `matplotlib` to `requirements.txt` to fix `Styler.background_gradient` crash - Formatted `src/dashboard.py` with Black/Ruff - Verified changes with Playwright screenshot Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
|
👋 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 New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
- Added `help` text to sidebar inputs for better usability - Added `matplotlib` to `requirements.txt` to fix `Styler.background_gradient` crash - Formatted entire repository with Black and Ruff to satisfy CI checks - Resolved all F401 (unused import) errors in `src/modules/__init__.py` - Verified changes with Playwright and Pytest Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
1 participant
💡 What: Added
helptooltips to key sidebar inputs (e.g., High Volatility Quantile, Transaction Cost) and addedmatplotlibdependency.🎯 Why: Users unfamiliar with jargon like "bps" or "ADV" might misconfigure the backtest. Tooltips provide concrete examples (e.g., "10 bps = 0.10%").
🛠 Fix: The app was crashing due to
ImportError: Styler.background_gradient requires matplotlib. Addedmatplotlibtorequirements.txtto resolve this and enable dataframe styling.♿ Accessibility: Improved clarity for screen readers and new users by explaining complex financial terms inline.
📸 Verification: Verified via Playwright screenshot showing tooltip and functional dashboard.
PR created automatically by Jules for task 14187557975875818488 started by @aarjava