Prior tests
I updated the prior tests to test the specific priors we've implemented. It now tests that the rescale
, sample
and prob
functions work as expected with either float inputs or arrays.
I had to change the prob
functions in many cases to allow arbitrary length arguments.
Do we want these functions to work for lists? This is currently not implemented.
@moritz.huebner did I get testing right?
Merge request reports
Activity
mentioned in commit d9cba37f
@colm.talbot Hey just looking briefly through the tests I would just remark that you typically only have one 'assert' statement per test function. That makes it easier to trace down errors later on. Each function should usually contain the minimal code required to get to the 'assert' statement.
That will make the testing code likely much longer and repetitive (which is typically the case for testing code), but that is not an immediate concern.
Edited by Moritz Huebner