Skip to content
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

[pull] master from tinygrad:master #185

Merged
merged 2 commits into from
Mar 19, 2025
Merged

[pull] master from tinygrad:master #185

merged 2 commits into from
Mar 19, 2025

Conversation

pull[bot]
Copy link

@pull pull bot commented Mar 19, 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 : )

chenyuxyz and others added 2 commits March 19, 2025 12:08
fixed `AssertionError: ' 1.00/10.0  1000it/s]' != ' 1.00/10.0  1.00kit/s]'`
* wow argmax is so good

* 1 less line

* clean up and better variable names

* is this torch thing right...?

* add more tests

* slap a TODO on it

* clean ups

* prettier looking code and fix ceil mode test

* add return types and some docs

* ok that was a bad example since indices == value, just no example
@pull pull bot added the ⤵️ pull label Mar 19, 2025
@pull pull bot merged commit 8c0d0a1 into kp-forks:master Mar 19, 2025
Copy link

Changes

Name                   Lines    Diff    Tokens/Line    Diff
-------------------  -------  ------  -------------  ------
tinygrad/tensor.py      1321      +8           20.3    -0.0
tinygrad/helpers.py      294      +1           15.9    -0.0


total lines changes: +9

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