-
Notifications
You must be signed in to change notification settings - Fork 3
Feat/test devices #74
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
base: develop
Are you sure you want to change the base?
Conversation
|
Already fixed many failures, but here's a log of the remaining failing tests... Testing locally, therefore for the following combinations
Test status: extreme
solar
vertical
score
stats
thermo
|
|
The mps issues seem unrelated to the code and might be an issue we can't fix here. Since torch-cpu is passing, I have decided for now to ignore the issues with mps with a view to revisiting later. Will test cuda-based backend/device combos instead |
|
Tests are passing for all the following combos*
*with some exceptions that we skip for now
|
Close #73
Close #65
Description
Test devices as well as backends. Uses eku
refactor/backend_for_testing_onlybranch.Issues to note, but finally decided not to tackle here:
Other points:
For now, I decided to use directly the namespaces and not the backends (I added array-api-compat versions of
xp.iscloseandxp.allclose, but we may want to reconsider that later if we want to have e.g. default dtype per device handling)Contributor Declaration
By opening this pull request, I affirm the following: