Skip to content

Make SVM visualization mesh step size configurable - #18

Closed
AdrienSkr with Copilot wants to merge 2 commits into
svm_classifierfrom
copilot/sub-pr-13-again
Closed

Make SVM visualization mesh step size configurable#18
AdrienSkr with Copilot wants to merge 2 commits into
svm_classifierfrom
copilot/sub-pr-13-again

Conversation

Copilot AI commented Dec 13, 2025

Copy link
Copy Markdown

The mesh grid step size in plot_decision_boundary() was hardcoded as 0.02, making it difficult to adjust visualization resolution for different use cases.

Changes:

  • Added mesh_step_size: 0.02 parameter to config.yaml under models.svm
  • Added optional step_size parameter to plot_decision_boundary() method
  • Falls back to config value when not specified, with final default of 0.02 if config missing
  • Updated docstring to explain resolution impact

Usage:

# Use default from config
svm.plot_decision_boundary(X, y)

# Override for higher resolution
svm.plot_decision_boundary(X, y, step_size=0.01)

# Or configure globally in config.yaml
models:
  svm:
    mesh_step_size: 0.01  # Higher resolution plots

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: Baddsu51 <85847277+Baddsu51@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback from review on SVM visualization changes Make SVM visualization mesh step size configurable Dec 13, 2025
Copilot AI requested a review from AdrienSkr December 13, 2025 15:30
@AdrienSkr AdrienSkr closed this Dec 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants