Change dimension ordering #50
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Necessary because chain_inds are not identical to those in the previous example (now repeating with inner instead of outer)
Pull Request Test Coverage Report for Build 3679875727Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
Codecov ReportBase: 94.41% // Head: 94.50% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #50 +/- ##
==========================================
+ Coverage 94.41% 94.50% +0.09%
==========================================
Files 9 9
Lines 609 619 +10
==========================================
+ Hits 575 585 +10
Misses 34 34
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
I opted to wait to allow |
|
Bump @devmotion do you think you will have time to review this soon? |
devmotion
left a comment
devmotion
left a comment
There was a problem hiding this comment.
Sorry @sethaxen, I missed that the PR was waiting for my review. Looks mostly good 👍 I had some minor comments and questions but there shouldn't be any major issues.
Did you rerun #49 (comment) with the updated ordering?
Co-authored-by: David Widmann <devmotion@users.noreply.github.com>
|
@devmotion I have implemented all requested changes. |
| Pkg.develop(; path=dirname(dirname(pathof(MCMCDiagnosticTools)))) | ||
| Pkg.instantiate() | ||
| include(joinpath("rstar", "runtests.jl")) | ||
| if Sys.WORD_SIZE == 64 |
There was a problem hiding this comment.
Since we do not only test XGBoost but also e.g. SVC nowadays maybe this check could be moved to rstar.jl and only XGBoost could be disabled. Not part of this PR though, and I guess nobody cares since we only run tests on 64bit anyway.
Co-authored-by: David Widmann <devmotion@users.noreply.github.com>
Changes dimension ordering for 3d arrays to be
(nparams, ndraws, nchains)(ndraws, nchains, nparams)andadds method with 3d array inputs foras proposed in #49.mcse