convert streamlit app example#131
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds a new example demonstrating how to convert a Streamlit application to a Panel application using Claude Code as an AI assistant.
Changes:
- Adds new documentation example showing the conversion of the Streamlit Stock Peers demo app to Panel
- Updates Panel and Panel-Material-UI skill documentation with UI component best practices
- Adds codespell configuration to handle "ACN" stock ticker
- Updates example documentation to use more professional language
Reviewed changes
Copilot reviewed 6 out of 10 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/holoviz_mcp/config/resources/skills/panel.md | Added guidance about making sidebar components stretch width |
| src/holoviz_mcp/config/resources/skills/panel-material-ui.md | Added best practices for Material UI icons and sidebar component sizing; removed blank line |
| pyproject.toml | Added codespell configuration to ignore "acn" (stock ticker) and skip binary/config files |
| mkdocs.yml | Added navigation entry for the new "Convert Streamlit App" example |
| docs/examples/generate-plot-from-picture.md | Updated result description to use more professional language |
| docs/examples/convert-streamlit-app.md | New comprehensive example with code, images, and step-by-step instructions for converting a Streamlit app to Panel |
| docs/assets/images/examples/stock-peer-analysis-streamlit-claude.png | New screenshot showing Claude Code prompt interface |
| DEFAULT_STOCKS = ["AAPL", "MSFT", "GOOGL", "NVDA", "AMZN", "TSLA", "META"] | ||
|
|
||
| HORIZON_MAP = { | ||
| "1 Months": "1mo", |
There was a problem hiding this comment.
Inconsistent grammar: "1 Months" should be "1 Month" (singular form when the number is 1).
| "1 Months": "1mo", | |
| "1 Month": "1mo", |
|
|
||
| self.horizon_selector = pmui.RadioButtonGroup( | ||
| label="Time horizon", | ||
| options=["1 Months", "3 Months", "6 Months", "1 Year", "5 Years", "10 Years", "20 Years"], |
There was a problem hiding this comment.
Inconsistent grammar: "1 Months" should be "1 Month" (singular form when the number is 1). This is consistent with the HORIZON_MAP definition above.
| options=["1 Months", "3 Months", "6 Months", "1 Year", "5 Years", "10 Years", "20 Years"], | |
| options=["1 Month", "3 Months", "6 Months", "1 Year", "5 Years", "10 Years", "20 Years"], |
| - DO use Material UI `sx` parameter for all css styling over `styles` and `stylesheets` | ||
| - DO use panel-material-ui components instead of panel components for projects already using panel-material-ui and for new projects | ||
| - DON'T configure the `design`, i.e. DO NOT `pn.extension(design='material')`. | ||
| - DO prefer professional Material UI icons over emojies |
There was a problem hiding this comment.
The word "emojies" should be "emojis" (standard plural form).
| - DO prefer professional Material UI icons over emojies | |
| - DO prefer professional Material UI icons over emojis |
|
|
||
| </details> | ||
|
|
||
| It even created passing tests, requirements file, and detailed documentation: |
There was a problem hiding this comment.
Inconsistent grammar: should be "It even created passing tests, a requirements file, and detailed documentation" to maintain parallel structure with the list items (all should be plural or singular consistently). Compare with the similar sentence in generate-dashboard-from-picture.md which says "It even created a tests, requirements.txt and a README.md" (which also has a grammar issue - "a tests" should be "tests").
| It even created passing tests, requirements file, and detailed documentation: | |
| It even created passing tests, a requirements file, and detailed documentation: |
No description provided.