Skip to content

refactor: Unified Sync/Async Tool Interface#438

Open
rayrishu19-wq wants to merge 1 commit into
sktime:mainfrom
rayrishu19-wq:refactor/unified-tool-interface
Open

refactor: Unified Sync/Async Tool Interface#438
rayrishu19-wq wants to merge 1 commit into
sktime:mainfrom
rayrishu19-wq:refactor/unified-tool-interface

Conversation

@rayrishu19-wq
Copy link
Copy Markdown

This PR addresses Issue #8 by merging separate sync and async tools into a single interface using a background parameter.

Changes:

  • Merged fit_predict_tool and fit_predict_async_tool in fit_predict.py.
    • Merged load_data_source_tool and load_data_source_async_tool in data_tools.py.
    • Updated server.py to use unified tool registration.
    • Updated test suite to align with new interface.

Verification:

  • All 132 tests passed locally.

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.

1 participant