🎨 Palette: Add help tooltips to financial parameters#53
🎨 Palette: Add help tooltips to financial parameters#53
Conversation
- Added `help` arguments to st.slider and st.number_input for: - Factor Beta Window - Regime Vol Window - ADV Participation % - High Volatility Quantile - Transaction Cost (bps) - Alert Thresholds (Drawdown, Volatility, Beta, DTTL) - Added matplotlib to requirements.txt for pandas styling support. - Formatted src/dashboard.py with black. 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` tooltips to financial inputs in `src/dashboard.py` (e.g., Transaction Cost, ADV %). - Formatted `src/` and `tests/` directories with `black` to fix CI lint failures. - Added `matplotlib` to `requirements.txt`. - Added UX journal entry. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
- Ran `ruff check --fix` to sort imports and fix linting errors (F401, C408, F841). - Updated `src/modules/__init__.py` to use explicit re-exports to satisfy Ruff F401. - Re-formatted codebase with `black` to match Ruff changes. - Verified local tests and lint checks pass. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
1 participant
💡 What: Added explanatory tooltips to 8 financial configuration inputs in the sidebar (e.g., Transaction Cost, ADV Participation, Volatility Regimes).
🎯 Why: Reduces cognitive load for users by defining domain-specific jargon like "basis points" and "quantiles" directly in the UI.
📸 Before/After: Visual inspection confirmed tooltips appear correctly.
♿ Accessibility: Improved understanding for non-expert users.
Notes: Also added matplotlib to requirements.txt as it is required by pandas styling.
PR created automatically by Jules for task 5285342545751103309 started by @aarjava