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

Unable to compute transpose of arrays with certain data types. #20

Closed
zoj613 opened this issue Jul 5, 2024 · 0 comments · Fixed by #41
Closed

Unable to compute transpose of arrays with certain data types. #20

zoj613 opened this issue Jul 5, 2024 · 0 comments · Fixed by #41
Labels
bug Something isn't working codecs relating to codecs help wanted Extra attention is needed

Comments

@zoj613
Copy link
Owner

zoj613 commented Jul 5, 2024

When using data types other than Complex32, Float32, Float64 and Complex64, The computation of an array's transpose fails with an unsupported operation error.

This isn't an issue with the correctness of the implementation but a weakness of the underlying n-dimensional array library we use: https://github.com/owlbarn/owl.

See the github issue tracking this bug: owlbarn/owl#671

This bug with Owl also affected the storage set_array function until I implemented a workaround that was added in 3523e879ececcde650f9b89e07db11cb7e1bf313

We have two options: To wait until the Owl library fixes this bug, or implement our own array transpose (that will be slower, surely) but very usable.

Note that this issue does not affect codec chains that do not contain that transpose codec.

@zoj613 zoj613 added bug Something isn't working help wanted Extra attention is needed codecs relating to codecs labels Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working codecs relating to codecs help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant