NeuralEnsemble / elephant / 20713099847

88%

Build Type

push

github

Committed by web-flow

Commit Message

[Fix] "too many resources requested for launch" error in the ASSET joint probability matrix computation when using CUDA (#667)

* Implemented parameter to override number of threads in PMatNeighbors

* Determine the number of threads in PMatNeighbors automatically or use the override

* Added logging functionality for debugging

* Added documentation on the new behavior for the number of threads in PMatNeighbors

* Updated logging statements to avoid errors

* Updated logging statements for simplified access to kernel parameters

* Added unit test for computing PMatNeighbors with varying thread numbers

* Added restriction to override number to avoid exceeding the maximum number of threads

* Removed one parameter iteration as this is currently expected to fail

* Added unit test for the high level cuda_threads parameter in the ASSET class

* Added missing reasons for skipping

* Added missing rate variable

* Added cleanup if variables were not originally set

* Added validation for cuda_thread parameter

* Test cases not needed as tuple is enforced to be (int, int)

* Added invalid combinations for the revised validation

* Not allowing None in the tuple

* Setting upper bound for number of threads per block

* Complementary unit tests for upper bound

---------

Co-authored-by: Cristiano Köhler <c.koehler@fz-juelich.de>
Co-authored-by: Harris Jos <104043391+CozySocksAlways@users.noreply.github.com>

Run Details

7 of 31 new or added lines in 1 file covered. (22.58%)

1 existing line in 1 file now uncovered.

6712 of 7624 relevant lines covered (88.04%)

5.26 hits per line