Skip to content

[pull] master from tinygrad:master#194

Merged
pull[bot] merged 3 commits intodandycheung:masterfrom
tinygrad:master
May 14, 2025
Merged

[pull] master from tinygrad:master#194
pull[bot] merged 3 commits intodandycheung:masterfrom
tinygrad:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented May 14, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

geohot and others added 3 commits May 13, 2025 16:51
* Make `dev` a property of `Allocator`

(this is a prereq refactor for #10285)

At least `BufferXfer.copy` accesses it assuming it's always present,
currently most devices just add this property on their own repeating
the same code over and over again.

This is also a bit footguny, see `RemoteAllocator` that named this
property `device` instead of `dev`, i could obviously just change that
in one place but doing it globally seems like a better solution (and it
reduces code duplication too).

`MallocAllocator` is a bit special, but passing `None` works just fine.

* typing

* ignore type instead of cast
@pull pull bot added the ⤵️ pull label May 14, 2025
@pull pull bot merged commit f726f79 into dandycheung:master May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants