Skip to content

[ENH] Add get_estimator_params MCP tool for inspecting fitted estimator parameters#379

Open
Vaishnav88sk wants to merge 1 commit into
sktime:mainfrom
Vaishnav88sk:feat/get-estimator-params
Open

[ENH] Add get_estimator_params MCP tool for inspecting fitted estimator parameters#379
Vaishnav88sk wants to merge 1 commit into
sktime:mainfrom
Vaishnav88sk:feat/get-estimator-params

Conversation

@Vaishnav88sk

Copy link
Copy Markdown

…or parameters

Reference Issues/PRs

Fixes #375

What does this implement/fix? Explain your changes.

What this PR does

  • Adds src/sktime_mcp/tools/inspect_estimator.py - new tool module with get_estimator_params_tool()
  • Registers the tool in server.py (both list_tools and call_tool)
  • Updates tools/init.py exports
  • Adds unit tests in tests/test_inspect_estimator.py

Does your contribution introduce a new dependency? If yes, which one?

What should a reviewer concentrate their feedback on?

Any other comments?

PR checklist

For all contributions
  • I've added myself to the list of contributors.
  • Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
  • I've added unit tests and made sure they pass locally.
For new estimators
  • I've added the estimator to the online documentation.
  • I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

@Vaishnav88sk

Copy link
Copy Markdown
Author

@Shashankss1205

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.

[ENH] Add get_estimator_params tool to inspect fitted estimator state

1 participant