Skip to content

[WIP] Add Pipeline with nested get_params and set_params - #9

Closed
akifkulll with Copilot wants to merge 1 commit into
mainfrom
copilot/add-pipeline-with-nested-params
Closed

akifkulll with Copilot wants to merge 1 commit into
mainfrom
copilot/add-pipeline-with-nested-params

Conversation

Copilot AI commented Oct 17, 2026 •

Copy link
Copy Markdown
  • Add Pipeline in /home/runner/work/PreToolkit/PreToolkit/src/preprocessing_toolkit/pipeline.py as a BasePreprocessor subclass with lazy validation in fit
  • Override nested get_params / set_params behavior (steps, step replacement, <name>__<param>) with atomic error handling
  • Implement chaining for fit, transform, and get_feature_names_out, including fit-failure unfitted behavior
  • Export Pipeline from /home/runner/work/PreToolkit/PreToolkit/src/preprocessing_toolkit/__init__.py
  • Add focused tests in /home/runner/work/PreToolkit/PreToolkit/tests/test_pipeline.py covering naming, validation, chaining, sklearn clone/grid-search/cross-val integration, and failure semantics
  • Run targeted tests first, then ruff check, ruff format --check, and full pytest
  • Run secret scanning on changed files and final parallel validation

Copilot AI linked an issue Oct 17, 2026 that may be closed by this pull request
10 tasks done
Copilot AI requested a review from akifkulll October 17, 2026 17:40
@ata-sesli ata-sesli closed this Oct 17, 2026
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.

Add Pipeline with nested get_params / set_params

3 participants