ReneEnjilian commented on PR #2271: URL: https://github.com/apache/systemds/pull/2271#issuecomment-2993730229
Hi @phaniarnab, I think there is a misunderstanding. With rewriting I dont mean CUDA kernels (defined in SystemDS.cu) but actually the Java methods in in `DoublePrecisionCudaSupportFunctions.java`. The same ones I changed in the first place. I only had to make changes to the files with deprecated methods (the ones I worked in the first place). A good example of what I mean with rewriting is that `cusparseSpGEMM` can not handle transpositions and needs sorted rows. So I had to find workarounds like writing my own transpose method for CSR matrices and methods that would ensure sorted rows on the GPU. There are no broken kernels. You can look at my changed files and will see what I mean. Further, I ran the unit tests that failed here. They both passed on my machine and the address SPARK and CP not GPU. So I would assume that this is a Cloud issue ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@systemds.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
