jblas-project / jblas

Linear Algebra for Java
http://jblas.org
BSD 3-Clause "New" or "Revised" License
590 stars 149 forks source link

org.jblas.TestDoubleMatrix::testColumnAndRowMinMax has inadequate loop coverage #92

Closed testingsavvy closed 6 years ago

testingsavvy commented 7 years ago

Currently, org.jblas.TestDoubleMatrix::testColumnAndRowMinMax() only exercises the loop in org.jblas.DoubleMatrix::argmin() such that the the element at index 0 is always returned as the min, although there is no specification for an input array that is sorted in ascending order. Similar argument for the other min functions in that test case. This PR adds org.jblas.TestDoubleMatrix::testColumnAndRowMinMaxUnsorted() to increase loop coverage, and find the min when the input array is unsorted.

mikiobraun commented 6 years ago

Thanks!