[WIP]MAHOUT-1974 (dense cuda multiplication) #318
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
andy@micheal:~/sandbox/mahout_cuda/cuda$ export SIZE_N=5000 andy@micheal:~/sandbox/mahout_cuda/cuda$ export SIZE_M=5000 andy@micheal:~/sandbox/mahout_cuda/cuda$ export SIZE_S=5000 andy@micheal:~/sandbox/mahout_cuda/cuda$ export DENSITY=.2 andy@micheal:~/sandbox/mahout_cuda/cuda$ export NUM_RUNS=2 andy@micheal:~/sandbox/mahout_cuda/cuda$ export SEED=1234
… into AP-UNIT-TEST
…nto MAHOUT-1974
…ly ending branch here
… needs to repacked mahout in-core matrices (row-major) as column major per the wrapped call to NVIDIA cublasDgemm operation. The fact that cusparse uses CSR rather than CSC is somewhat confusing in this case (for e.g. a Dense %*% Sparse JVM level operation
… need to use transposes. need to check values in unit tests, and transpose in dgemm call as needed
|
need to rebase |
andrewpalumbo
force-pushed
the
MAHOUT-1974
branch
from
dce10d7 to
961aee9
Compare
|
@nsakharnykh @pat @rawkintrevo FYI |
Fixed dense-dense path for JCuda, also switched to JCublas2 interface
|
@nsakharnykh sorry for the state of this branch, I tend to commit a lot on this project, and leave a lot of [WIP]s in when jumping around to other branches. Resolved all conflicts the against CUDA branch.. going to start looking at Dense %*% sparse again. First need to read up on |
|
@nsakharnykh I think that I'll fix the routing on this and test the sorted CSC requirement (MAHOUT-2004 and MAHOUT-2003) and then after some cleaning on this, we should push to mahout/CUDA. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of PR:
Please give a short description of what this PR is for.
Important ToDos
Please mark each with an "x"
If all of these things aren't complete, but you still feel it is
appropriate to open a PR, please add [WIP] after MAHOUT-XXXX before the
descriptions- e.g. "MAHOUT-XXXX [WIP] Description of Change"
Does this change break earlier versions?
Is this the beginning of a larger project for which a feature branch should be made?