Skip to content

Fix data hyper-cleaning pytest script path - #36

Open
alceops wants to merge 1 commit into
callous-youth:mainfrom
alceops:alce/fix-32-data-hyper-cleaning-test-path
Open

Fix data hyper-cleaning pytest script path#36
alceops wants to merge 1 commit into
callous-youth:mainfrom
alceops:alce/fix-32-data-hyper-cleaning-test-path

Conversation

@alceops

@alceops alceops commented Apr 29, 2026

Copy link
Copy Markdown

Fixes #32.

Summary

  • Replace the hard-coded GitHub Actions workspace path in the data hyper-cleaning pytest wrapper with a path derived from Path(__file__).
  • Use sys.executable so the example test runs with the same Python interpreter/virtualenv that invoked pytest.
  • Keep the existing parametrized coverage and CLI arguments unchanged.

Verification

  • python3 -m py_compile examples/data_hyper_cleaning/test_data_hyper_cleaning.py
  • git diff --check
  • Static assertion that /home/runner/work/BOAT is no longer present in the test file and the script path is repo-relative
  • /tmp/boat-venv/bin/python -m pytest examples/data_hyper_cleaning/test_data_hyper_cleaning.py --collect-only -q (68 tests collected)

Not fully run locally: executing the example currently needs the project ML dependencies; in this minimal verification venv, the first selected test stops at the pre-existing missing torch import. This PR only changes the test wrapper path/interpreter selection.

No private data, paid compute, GPU, secrets, deploy, account setup, or production access used.

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.

[BUG] data_hyper_cleaning pytest uses hard-coded GitHub Actions path

1 participant