Skip to content

[NA] [SDK] chore: remove unused LocalRunnerTask class#6037

Merged
petrotiurin merged 2 commits intomainfrom
petrotiurin/NA-remove-local-runner-task
Apr 7, 2026
Merged

[NA] [SDK] chore: remove unused LocalRunnerTask class#6037
petrotiurin merged 2 commits intomainfrom
petrotiurin/NA-remove-local-runner-task

Conversation

@petrotiurin
Copy link
Copy Markdown
Contributor

@petrotiurin petrotiurin commented Apr 1, 2026

Details

Removes LocalRunnerTask from the Python SDK. This class is not used and should not be a part of the SDK.

  • Deleted sdks/python/src/opik/evaluation/local_runner_task.py
  • Removed LocalRunnerTask from opik.evaluation.__all__ and opik.__all__
  • Deleted unit tests (tests/unit/evaluation/test_local_runner_task.py)
  • Deleted e2e tests (tests/e2e/runner/test_local_runner_task_e2e.py)

Change checklist

  • User facing
  • Documentation update

Issues

  • Resolves #
  • OPIK-

AI-WATERMARK

AI-WATERMARK: yes

  • If yes:
    • Tools: Claude Code
    • Model(s): claude-sonnet-4-6
    • Scope: full implementation
    • Human verification: code review + unit tests (2552 passed)

Testing

  • Ran full Python SDK unit test suite: pytest tests/unit/ -x -q — 2552 tests passed, 0 failed
  • Verified no remaining references to LocalRunnerTask in the codebase
  • make precommit-sdks passed (ruff, mypy, formatting)

Documentation

N/A

Removes LocalRunnerTask and all associated tests as this class is no longer used.

- Delete sdks/python/src/opik/evaluation/local_runner_task.py
- Delete unit and e2e tests for LocalRunnerTask
- Remove LocalRunnerTask from evaluation/__init__.py and opik/__init__.py exports
@github-actions github-actions bot added python Pull requests that update Python code tests Including test files, or tests related like configuration. Python SDK labels Apr 1, 2026
Comment thread sdks/python/src/opik/__init__.py
@petrotiurin petrotiurin marked this pull request as ready for review April 1, 2026 15:48
@petrotiurin petrotiurin requested a review from a team as a code owner April 1, 2026 15:48
Copy link
Copy Markdown
Member

@andrescrz andrescrz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please review failing tests before moving forward.

@petrotiurin
Copy link
Copy Markdown
Contributor Author

LGTM, please review failing tests before moving forward.

Thanks! Looks like there is some flakiness on the third-party integration side but it's unrelated to this change.

@petrotiurin petrotiurin merged commit 0a7bb67 into main Apr 7, 2026
128 of 132 checks passed
@petrotiurin petrotiurin deleted the petrotiurin/NA-remove-local-runner-task branch April 7, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Python SDK python Pull requests that update Python code tests Including test files, or tests related like configuration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants