🎨 Palette: Add tooltips to technical inputs in dashboard#45
🎨 Palette: Add tooltips to technical inputs in dashboard#45
Conversation
- Added `help` tooltips to inputs with financial jargon (bps, ADV, Quantile). - Created `.Jules/palette.md` to document UX learnings. - Fixed a crash by adding `matplotlib` to `requirements.txt` (required for pandas styling). 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. |
- Ran `ruff check --fix` on `src/dashboard.py` to fix literal dict syntax and trailing whitespace. - Ran `black src/dashboard.py` to format the file according to project standards. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
- Ran `black .` to format all Python files in the repository, resolving CI check failures. - Ran `ruff check . --fix` to resolve remaining linting issues, including adding `# noqa: F401` to `src/modules/__init__.py`. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
1 participant
This PR adds helpful tooltips to several technical inputs in the Streamlit dashboard, such as "Transaction Cost (bps)", "ADV Participation %", and "High Volatility Quantile". This helps users understand the parameters better without needing deep domain knowledge.
Also includes a fix for a missing
matplotlibdependency that caused the app to crash when rendering dataframe gradients.PR created automatically by Jules for task 5920252018871938830 started by @aarjava