Fixes to test suite to support CUDA arrays#118
Fixes to test suite to support CUDA arrays#118gaurav-arya wants to merge 5 commits intoJuliaMath:masterfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #118 +/- ##
==========================================
+ Coverage 93.70% 93.73% +0.02%
==========================================
Files 5 5
Lines 445 447 +2
==========================================
+ Hits 417 419 +2
Misses 28 28 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
gaurav-arya
left a comment
There was a problem hiding this comment.
Remove Random.rand! usage
197ca26 to
31755f0
Compare
|
Bump on review:) It should hopefully be an easy review, especially because this does not touch the core functionality, only the test utilities, which I have verified continue to work with |
…s for wrapper tests" This reverts commit e46b8eb.
|
Bump |
|
Bump on review. @devmotion raised in #118 (comment)) the issue of |
Implemented changes:
test_plan_adjoint*(see Handling wrapper array inputs #119. I don't expect it to be solved in the immediate future, which is why I prefer to work around it for now).fftdimsas an iterable rather than its specific form, in accordance with the function contract.