-
Notifications
You must be signed in to change notification settings - Fork 359
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
Tinygrad support #297
Tinygrad support #297
Conversation
70aed73
to
789d1fd
Compare
bc07359
to
b916f7f
Compare
b916f7f
to
970ed5a
Compare
|
tinygrad/tinygrad#2686 merged, ready for review 🖖 |
@arogozhnikov Is it possible to merge this? |
I wish you folks implemented array api instead - and got E.g. in |
@arogozhnikov We are interested in being array API compatible. Totally understand not writing special backend code Is there a good compliance test for this? How far off are we? |
@geohot to see how far you off, start with this additional method in def __array_namespace__(self, api_version: Optional[str] = None) -> Any:
return Tensor # tensor class, as it seems to contain most of relevant methods. I expect that due to divergence it would be easier to have another namespace like In a perfect world |
No description provided.