Conversation
Added `help` tooltips to `High Volatility Quantile`, `Transaction Cost`, and `ADV Participation` inputs to clarify their meaning and units (e.g., basis points) for non-expert users. Also ran linting and formatting on `src/dashboard.py`. 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 `High Volatility Quantile`, `Transaction Cost`, and `ADV Participation` inputs to clarify parameters. - Reformatted the entire codebase with `black` and `ruff` to fix CI failures (linting and formatting checks). - Added a UX journal entry to `.Jules/palette.md` regarding financial input accessibility. This commit addresses both the UX enhancement request and the repository-wide formatting standards required by CI. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
- Added `help` tooltips to `High Volatility Quantile`, `Transaction Cost`, and `ADV Participation` inputs. - Reformatted the entire codebase with `black` and `ruff` to fix CI failures. - Added `# noqa: F401` to `src/modules/__init__.py` to resolve unused import errors during CI checks. - Added UX journal entry to `.Jules/palette.md`. This commit finalizes the UX enhancement while ensuring full compliance with the project's strict CI linting and formatting rules. Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
1 participant
💡 What: Added explanatory tooltips to complex financial inputs in the sidebar.
🎯 Why: Users may not understand terms like "bps" (basis points) or how the "High Volatility Quantile" affects the model. Providing context improves accessibility and usability.
📸 Before/After: Inputs now display a help icon (?) which shows a tooltip on hover explaining the parameter.
♿ Accessibility: Improves cognitive accessibility by reducing jargon barrier.
PR created automatically by Jules for task 1405259981735683624 started by @aarjava