-
Notifications
You must be signed in to change notification settings - Fork 18
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
reshape
function
#558
Comments
This is exactly what I'd like to avoid -- |
I believe we can do this. The main challenge I have a hard time designing for is the selection of an appropriate output format. If we know the appropriate output format, I don't think it would be so hard to do a direct copy. We would need to use one of the randomly accessible sparse formats if the swizzles don't match up. I think we could implement something similar to the style of our getindex kernel, where we use a generated function to write some Finch code and call it. |
@mtsokol Is this is a blocker for you? |
@willow-ahrens It's needed for doing finch-tensor/finch-tensor-python#48 cleanly. |
hmm. Okay! I'll do it today then. |
Hi @willow-ahrens,
I wanted to discuss feasibility of eager only (and simplest possible)
reshape
function in Finch.It looks like most of the Array API test suites require
reshape
to run (test_signatures.py
test suite that I ran to count the number of supported functions is an exception).To run them locally I used a crude/test only implementation:
Do you think it would be feasible to have an eager only
reshape(::SwizzleArray, ...)
? Even if it would be as simple as copy to a dense format ([EDIT] or COO?), reshape and then copy back to the original format?The text was updated successfully, but these errors were encountered: