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

Adds unstack #831

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

Conversation

emchristiansen
Copy link

  1. Adds an empty .rustfmt.toml so formatting for this project will be respected in workspaces with different formatting.
  2. Adds unstack, which is the inverse of stack.

Note: I've never before made such liberal use of the Rust type system, so I basically did a lot of pattern-matching with stack to figure out what to do.
You'll probably want to review it carefully.
(The tests do pass, though.)

Note: I'm not confident in my ability to write correct CUDA kernels using your library, so I only implemented the CPU backend.

@emchristiansen emchristiansen changed the title Eric0000 refactor unstack Adds unstack Jul 26, 2023
@emchristiansen
Copy link
Author

Hey, is this something you're interested in?
FYI I've been using this fn and it appears to work.

@swfsql swfsql mentioned this pull request Feb 6, 2024
6 tasks
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