-
Notifications
You must be signed in to change notification settings - Fork 1
Running tests in CI #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
OK, yes, so indeed the Pyodide venv currently doesn't work with scipy due to pyodide/pyodide#3186 I think. This successfully builds the wheel for 1.1.1 , but that's not very useful since we are for now not loading it in the included test scripts (and it's the same version anyway as the one shipped by Pyodide 0.22.0a1). For scikit-learn main branch, there is some regression with omp.h include that fails to cross-compile. Byt anyway I agree that the first step could be looking into fixing tests for 1.1.1. I think some number of the failures would be due to pyodide/pyodide#156 |
|
I'll merge this one when CI is green, this is good enough to be useful. Possible improvements:
|
Also add tested module when timing out
Related #1
Edit: close #1 since this is now building scikit-learn from main.