-
-
Notifications
You must be signed in to change notification settings - Fork 183
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
Rev for Cholesky on GPU #1118
Merged
Merged
Rev for Cholesky on GPU #1118
Changes from 1 commit
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
15be4dd
Moved over main rev function to branch and did simple cleaning
SteveBronder 0196022
Moves Cholesky GPU into the cholesky file
SteveBronder ff32352
Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev
SteveBronder 463fb32
Merge branch 'gpu-cholesky-rev' of https://github.com/bstatcomp/math …
SteveBronder 4f156f4
Update rev cholosky to use new gpu functions
SteveBronder 37d5b18
Fixup rev code
SteveBronder 0d88984
update rev
SteveBronder ac03052
reverting the copy_tri kernel changes
rok-cesnovar a77afda
include header remove
rok-cesnovar 66799ba
Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev
rok-cesnovar b87dffa
minor comments
rok-cesnovar 37075df
Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev
rok-cesnovar 2718f05
added rev opencl tests, failing
rok-cesnovar af7f16a
Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev
rok-cesnovar 69e7d9c
further work on rev, still fails
rok-cesnovar 5f0cc21
Update tests, now passing
SteveBronder 1e27c13
missing ifdef in rev, caused fails without STAN_OPENCL
rok-cesnovar 6105b19
fixed the multiply(Nx0, 0xM) bug
rok-cesnovar 0363c08
removed the size>0 check
rok-cesnovar c56c5d7
Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev
rok-cesnovar cd65c9b
sets the tests to go off on the GPU for smaller cholesky sizes
SteveBronder eb33397
Merge remote-tracking branch 'origin/gpu_cholesky_prim' into gpu-chol…
SteveBronder b95beba
Merge commit 'a933f65c4f584fb2ff949b350816eae59b72ddfd' into HEAD
yashikno 0bf89b4
[Jenkins] auto-formatting by clang-format version 5.0.0-3~16.04.1 (ta…
stan-buildbot a3c1650
Merge remote-tracking branch 'upstream/develop' into gpu-cholesky-rev
SteveBronder a82d581
Merge branch 'gpu-cholesky-rev' of https://github.com/bstatcomp/math …
SteveBronder 96b2a40
Accidentally took out some tests that were supposed to be there
SteveBronder f2bb7c4
fix some simple math
SteveBronder 8aa02ee
Adds bug fix so that GPU cholesky only works on type double. Also mak…
SteveBronder 8aa1c10
Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev
rok-cesnovar c02c813
Merge branch 'develop' into gpu_cholesky_prim
rok-cesnovar 4aa3029
Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev
rok-cesnovar 9df78a4
[Jenkins] auto-formatting by clang-format version 5.0.0-3~16.04.1 (ta…
stan-buildbot 316ba55
Revert "Merge branch 'gpu_cholesky_prim' into gpu-cholesky-rev"
rok-cesnovar d8b476a
addressing minor comments
rok-cesnovar decbb72
For cholesky_opencl in rev, Removes the M = M_, sets the inner loops …
SteveBronder 705ef7e
merge to remote
SteveBronder 9846f74
[Jenkins] auto-formatting by clang-format version 5.0.0-3~16.04.1 (ta…
stan-buildbot 59ffbf9
merge to develop
SteveBronder 7197dba
remove duplicate include
rok-cesnovar 32560f5
Merge commit 'f0e38e580d1d34d91fb202b0134d76e9d92a5226' into HEAD
yashikno c0aeaf6
Merge remote-tracking branch 'origin/develop' into feature/issue-1058…
SteveBronder bd7b13b
Cleans up L<var> assignment in cholesky_decompose. Removes unneeded g…
SteveBronder 671901b
Merge branch 'gpu-cholesky-rev' of https://github.com/bstatcomp/math …
SteveBronder 2514def
[Jenkins] auto-formatting by clang-format version 5.0.0-3~16.04.1 (ta…
stan-buildbot e8a4e48
Fixes the math in the block size calculation for cholesky_block and m…
SteveBronder 59aab65
merge to origin remote
SteveBronder 483ce25
Merge commit '2850ec262181075a5843968e805dc9ad1654e069' into HEAD
yashikno 95adc28
[Jenkins] auto-formatting by clang-format version 5.0.0-3~16.04.1 (ta…
stan-buildbot b92441a
Removes comment that was not truthful about not doing pos_def checks
SteveBronder bef42c7
check_symmetric with OpenCL in prim
rok-cesnovar 0e2ef93
check_symmetric removed in rev for the OpenCL version, its a duplicat…
rok-cesnovar 1e09d95
Merge remote-tracking branch 'upstream/develop' into gpu-cholesky-rev
SteveBronder 9037b8d
removing camelCase and unwanted underscores
rok-cesnovar 5c2e48d
added 2 tests below the threshold
rok-cesnovar c45bac0
bar goes adj
rok-cesnovar 5117236
symbolic_rev added, better zeroing
rok-cesnovar 1d17106
Merge commit 'c72d56f4d29d2cc5d3535ee433d7a9695fb681fc' into HEAD
yashikno d4c8646
[Jenkins] auto-formatting by clang-format version 5.0.0-3~16.04.1 (ta…
stan-buildbot d6d7fa0
Merge branch 'gpu-cholesky-rev' of https://github.com/bstatcomp/math …
SteveBronder d0c124f
Merge remote-tracking branch 'upstream/develop' into gpu-cholesky-rev
SteveBronder 9784cc2
outsource to prim cholesky directly
rok-cesnovar 0db6764
[Jenkins] auto-formatting by clang-format version 6.0.0 (tags/google/…
stan-buildbot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
bar goes adj
- Loading branch information
commit c45bac024851c176a75c44f88214c873a86e6928
There are no files selected for viewing
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't actually have to doc stuff in the
internal
namespace, but this is nice that you have :)