Skip to content

Skip signature feature tests when esig is unavailable - #3

Merged
steveya merged 2 commits into
claude/add-claude-documentation-4HfdFfrom
copilot/sub-pr-2
Mar 20, 2026
Merged

steveya merged 2 commits into
claude/add-claude-documentation-4HfdFfrom
copilot/sub-pr-2

Conversation

Copilot AI commented Mar 20, 2026 •

Copy link
Copy Markdown
Contributor

CI was failing with 14 ModuleNotFoundError errors in test_signature_features.py because esig — a hard runtime dependency of sktime's SignatureTransformer — is not installed in the CI environment.

Change

Added pytest.importorskip("esig") at module level in tests/test_signature_features.py, consistent with the existing pytest.importorskip("alphaforge") pattern:

pytest.importorskip("alphaforge")
pytest.importorskip("esig")

This causes the entire module to be skipped rather than erroring when esig is absent.


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Co-authored-by: steveya <7390489+steveya@users.noreply.github.com>
Copilot AI changed the title [WIP] Add CLAUDE.md project documentation Skip signature feature tests when esig is unavailable Mar 20, 2026
Copilot AI requested a review from steveya March 20, 2026 03:38
@steveya
steveya marked this pull request as ready for review March 20, 2026 03:44
@steveya
steveya merged commit b593535 into claude/add-claude-documentation-4HfdF Mar 20, 2026
@steveya
steveya deleted the copilot/sub-pr-2 branch March 20, 2026 03:44
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.

2 participants