Skip to content

Conversation

@Willy-Chan
Copy link
Collaborator

@simonguozirui We need to discuss software design decisions

@Willy-Chan
Copy link
Collaborator Author

Willy-Chan commented Dec 18, 2025

@simonguozirui @willhu-jpg I opened a new more simplified PR that uses torch load_inline and clones the TK repo to the modal. In my view that version is a lot better (much less code change, I can't think of too many advantages to this much more complicated approach). Let me know if I should close this!

This separate .cu/.py version is compatible with our old set of kernels we generated for CS 191W, I will keep it on my fork for legacy reasons in case we need it.

Link to the (better) PR: #107

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.

1 participant