search

mlpack / models

models built with mlpack
https://models.mlpack.org/docs
BSD 3-Clause "New" or "Revised" License
35 stars 40 forks source link

Use catch2 instead of ctest #70

Closed Aakash-kaushik closed 3 years ago

Aakash-kaushik commented 3 years ago

ctest reports tests to pass even if they are failed, this is done just so we know if some tests fail.

reference comment: https://github.com/mlpack/models/pull/63#issuecomment-870899105

reverting back on #69 closed it automatically so creating this PR.

Aakash-kaushik commented 3 years ago

the dir command in windows is just for debugging and will be removed before the final merge.

Aakash-kaushik commented 3 years ago

@zoq should be good to merge now.