Skip to content

Conversation

@tarekziade
Copy link
Contributor

Investigating CoreML crash that occurs in CI but not locally. Want to determine if the issue is Python version-specific.

tarekziade and others added 4 commits January 14, 2026 22:37
Investigating CoreML crash that occurs in CI but not locally.
Want to determine if the issue is Python version-specific.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The Makefile was hardcoded to use python3.12, which meant GitHub Actions
would set up the requested Python version (3.11, 3.13) but the venv would
always be created with 3.12. This caused version mismatches in CI.

Now using 'python3' which will use whatever Python is in the PATH, set by
the GitHub Actions setup-python step.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The CoreML crash in CI was because the original rustnn workflow only ran
comprehensive Python tests on Linux, where CoreML isn't available.

Changes:
- Split into python-build (all platforms) and python-test (Linux only)
- python-build: Verifies package builds and imports on all OS/Python versions
- python-test: Runs full test suite on Linux with ONNX backend
- macOS still builds with CoreML for wheel validation, but doesn't run tests
- Fixed Makefile to use python3 from PATH instead of hardcoded 3.12

This avoids the CoreML framework crash on GitHub Actions macOS runners while
maintaining comprehensive ONNX backend test coverage.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
pywebnn doesn't have a 'python' feature flag (unlike rustnn) since it's
already Python-only. Updated workflow and Makefile to use correct features:
- macOS: onnx-runtime,coreml-runtime
- Linux/Windows: onnx-runtime only

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@tarekziade tarekziade merged commit 6eb320a into main Jan 15, 2026
12 checks passed
@tarekziade tarekziade deleted the test/macos-py313-only branch January 15, 2026 07:48
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