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

btas::contract with Column major. #39

Open
shiozaki opened this issue Jun 15, 2014 · 2 comments
Open

btas::contract with Column major. #39

shiozaki opened this issue Jun 15, 2014 · 2 comments

Comments

@shiozaki
Copy link
Contributor

btas::contract(1.0, *this, {0,1,3}, *cmat, {3,2}, 0.0, *out, {0,1,2}); with the column major should not permute indices, but it does. Not sure if this is a problem in the index notation, or that of the implementation (or both).

@naokin
Copy link
Contributor

naokin commented Jun 15, 2014

I think just an implementation issue, it should be checked if permutation can be replaced by CblasTrans directive before calling BLAS in btas/generic/contract.h

@shiozaki
Copy link
Contributor Author

Naoki, in this case there is no transposition either (because it is column major), and I think that btas::contract is biased towards row major. I could look into this later (but I will be hosting a conference till Tuesday..)

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

No branches or pull requests

2 participants