dougshidong / PHiLiP

Parallel High-Order Library for PDEs through hp-adaptive Discontinuous Galerkin methods
Other
46 stars 37 forks source link

Tests `NNLS_dependent_columns` and `NNLS_multi_core` may fail #266

Open cpethrick opened 2 months ago

cpethrick commented 2 months ago

A seed is set for a random number generator at line 788 of tests/unit_tests/linear_solver/NNLS_tests.cpp. Some seeds may cause the test to fail.

tofstie commented 2 months ago

This test failure is the result of the matrices in Tests "NNLS_dependent_columns" and "NNLS_wide_matrix" being generated with no solution guaranteed. There are two approaches to fix this test failure.

  1. Find a random seed that does not fail
  1. In the Optimality function, set the random flag to true

Either one or both fixes can be done, and I can submit a PR once I know which to do.