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

Stochastic matmul verify #1134

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

andrej
Copy link
Contributor

@andrej andrej commented Mar 15, 2024

No description provided.

@jgmelber
Copy link
Collaborator

jgmelber commented Apr 1, 2024

Is this still a WIP or ready to merge?

@andrej
Copy link
Contributor Author

andrej commented Apr 1, 2024

Let me rebase this on refactor-matmul. Then, once refactor-matmul is merged, we can merge this if you are cool with it

@andrej andrej force-pushed the matmul-verify-stochastic branch 4 times, most recently from b62eca6 to 47c796f Compare April 3, 2024 19:48
@andrej andrej marked this pull request as ready for review April 5, 2024 17:54
@andrej
Copy link
Contributor Author

andrej commented Apr 5, 2024

This also contains the fixes for the verification code on the host (n_rows vs n_cols confusion). Ready for review! @jgmelber

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants