{"action":"created","issue":{"url":"https://api.github.com/repos/deeplearning4j/deeplearning4j/issues/6378","repository_url":"https://api.github.com/repos/deeplearning4j/deeplearning4j","labels_url":"https://api.github.com/repos/deeplearning4j/deeplearning4j/issues/6378/labels{/name}","comments_url":"https://api.github.com/repos/deeplearning4j/deeplearning4j/issues/6378/comments","events_url":"https://api.github.com/repos/deeplearning4j/deeplearning4j/issues/6378/events","html_url":"https://github.com/deeplearning4j/deeplearning4j/issues/6378","id":357265108,"node_id":"MDU6SXNzdWUzNTcyNjUxMDg=","number":6378,"title":"MMulTranspose not working correctly","user":{"login":"RSBat","id":6381474,"node_id":"MDQ6VXNlcjYzODE0NzQ=","avatar_url":"https://avatars2.githubusercontent.com/u/6381474?v=4","gravatar_id":"","url":"https://api.github.com/users/RSBat","html_url":"https://github.com/RSBat","followers_url":"https://api.github.com/users/RSBat/followers","following_url":"https://api.github.com/users/RSBat/following{/other_user}","gists_url":"https://api.github.com/users/RSBat/gists{/gist_id}","starred_url":"https://api.github.com/users/RSBat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RSBat/subscriptions","organizations_url":"https://api.github.com/users/RSBat/orgs","repos_url":"https://api.github.com/users/RSBat/repos","events_url":"https://api.github.com/users/RSBat/events{/privacy}","received_events_url":"https://api.github.com/users/RSBat/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-09-05T14:47:12Z","updated_at":"2018-09-06T05:51:54Z","closed_at":null,"author_association":"NONE","body":"`mmul ` with MMulTranspose argument gives wrong result if `transposeResult` is set to `true`\r\nI think the problem is here (this block looks strange anyways):\r\nhttps://github.com/deeplearning4j/deeplearning4j/blob/1c0fda4ceaf7ed9c1d174bb111779fcb1692ee33/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/blas/params/MMulTranspose.java#L53-L64\r\n\r\nEDIT: I also noticed that there is extra output in `mmul` function: https://github.com/deeplearning4j/deeplearning4j/blob/1c0fda4ceaf7ed9c1d174bb111779fcb1692ee33/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ndarray/BaseNDArray.java#L3418-L3419\r\n\r\nI think I can make a pull request with fix if you want"},"comment":{"url":"https://api.github.com/repos/deeplearning4j/deeplearning4j/issues/comments/418972912","html_url":"https://github.com/deeplearning4j/deeplearning4j/issues/6378#issuecomment-418972912","issue_url":"https://api.github.com/repos/deeplearning4j/deeplearning4j/issues/6378","id":418972912,"node_id":"MDEyOklzc3VlQ29tbWVudDQxODk3MjkxMg==","user":{"login":"saudet","id":3328023,"node_id":"MDQ6VXNlcjMzMjgwMjM=","avatar_url":"https://avatars1.githubusercontent.com/u/3328023?v=4","gravatar_id":"","url":"https://api.github.com/users/saudet","html_url":"https://github.com/saudet","followers_url":"https://api.github.com/users/saudet/followers","following_url":"https://api.github.com/users/saudet/following{/other_user}","gists_url":"https://api.github.com/users/saudet/gists{/gist_id}","starred_url":"https://api.github.com/users/saudet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saudet/subscriptions","organizations_url":"https://api.github.com/users/saudet/orgs","repos_url":"https://api.github.com/users/saudet/repos","events_url":"https://api.github.com/users/saudet/events{/privacy}","received_events_url":"https://api.github.com/users/saudet/received_events","type":"User","site_admin":false},"created_at":"2018-09-06T05:51:54Z","updated_at":"2018-09-06T05:51:54Z","author_association":"MEMBER","body":"@RSBat No, I didn't write that code. If you have a suggestion though, please do send it as a pull request yes. It will make it easier to have it reviewed by the right person."}}
{"actor":{"display_login":"saudet"}}