Closed jared321 closed 2 months ago
Unfortunately, I believe that this task is intimately tied to the MATLAB path issue #127. I will try to fix up paths as managed by POUNDERS and its tests to get this working.
Testpounders
just call individual scripts such as test_one_m_pounders
. The paths shall be setup automatically within each of these test/example executables so that a user can run MATLAB from the POUNDERS test folder and execute correctly the function by itself with no need for unnecessary additions to the MATLAB path. At the moment, the only necessary cases should be related to BenDFO which is a repo separate from IBDFO and not included in it as a submodule (as MINQ is).pounders
, pounders/m
, and pounders/m/tests
without needing to adjust paths. This implies that paths in the code and tests should not be relative to the current working directory. This requirement is inline with the fact that runtests
will run tests only in the current working directory but can be asked to look for tests in subfolders.
I have updated manifold sampling as well so that in theory it could be tested with coverage in the action as well. However, it has an external dependency to the allcomb
function in the MathWorks file exchange. To download this for working locally, I had to log in with a MathWorks username/password.
I would like to avoid having a CI action that logs into FileExchange with an account since that is more complex and I prefer to avoid potential security issues. A few other possibilities to explore:
Note that use of Projects/Toolboxes should be investigated simultaneously with the MATLAB path management Issue #127.
Merged into develop
. Closed.
Check to see if MATLAB tests are being automatically run in GitHub actions. If not, then implement this including (hopfully) code coverage.