docs: enhance matplotlib backend guidance in panel skills#104
Conversation
There was a problem hiding this comment.
Pull request overview
This PR enhances the documentation for using matplotlib with Panel by adding comprehensive guidance on backend configuration, extension usage, and best practices. Additionally, it corrects the tool namespace reference in the HoloViz Analysis Planner agent configuration.
Changes:
- Added new matplotlib section to Panel skills documentation with backend configuration guidance and code examples
- Updated agent configuration to use correct tool namespace pattern (
holoviz/*instead ofholoviz-dev/*)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/holoviz_mcp/config/resources/skills/panel.md | Added matplotlib section with guidance on 'agg' backend configuration, pn.extension() usage, and best practices for static plots |
| .github/agents/holoviz-analysis-planner.agent.md | Corrected tool namespace from 'holoviz-dev/' to 'holoviz/' to match actual MCP server tool naming convention |
|
|
||
| ### Matplotlib | ||
|
|
||
| **CRITICAL**: On windows set non-interactive backend before importing matplotlib.pyplot: |
There was a problem hiding this comment.
The guidance incorrectly suggests that setting the 'agg' backend is only necessary on Windows. The 'agg' backend should be set on all platforms when using matplotlib with Panel for server-side rendering, not just Windows. Consider revising to: "CRITICAL: Set non-interactive backend before importing matplotlib.pyplot:" or "CRITICAL: When using Panel with matplotlib, set non-interactive backend before importing matplotlib.pyplot:"
| **CRITICAL**: On windows set non-interactive backend before importing matplotlib.pyplot: | |
| **CRITICAL**: When using Panel with matplotlib, set a non-interactive backend (e.g. `'agg'`) before importing matplotlib.pyplot: |
|
|
||
| ### Matplotlib | ||
|
|
||
| **CRITICAL**: On windows set non-interactive backend before importing matplotlib.pyplot: |
There was a problem hiding this comment.
The word "windows" should be capitalized to "Windows" when referring to the operating system.
| **CRITICAL**: On windows set non-interactive backend before importing matplotlib.pyplot: | |
| **CRITICAL**: On Windows set non-interactive backend before importing matplotlib.pyplot: |
No description provided.