-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MATLAB Path Management #127
Comments
It looks like users (or perhaps the person who writes something like |
Some potentially useful info regarding
In light of this, is it acceptable for a tool like POUNDERS to blindly add |
pounders.m and its tests often add folders to the MATLAB path but do so using relative paths, which can influence from where you can call the code. The changes here adopt a different path strategy using paths relative to the file that is adding to the path. This is related to Issue #127. A second goal is to make it so that users don't have to add too many paths. Confirmed that the full test suite can be run as detailed in its inline documentation. This includes creating an HTML coverage report. Confirmed that changes to the path made by tests are not present upon termination. Ran each single test individually with a clean MATLAB environment to confirm that they run through with and only with the documented path requirements. Confirmed that they leave the path as found upon entry.
It looks like Naively, I would believe that if I want to use one method, I shouldn't add another method to the MATLAB path unless there is an obvious dependence such as between POUNDERS and MINQ. @jmlarson1 @mmenickelly |
As part of working Issue #145, I have come across
These might ease path management and overcome some of the aforementioned technical difficulties. Consensus is that for now we assume that MATLAB users are happy to work from a clone of the repo with the need for them to alter the path. In particular, there is no need to offer the MATLAB code as a potentially simpler, cleaner toolbox through FileExchange. The possibility of methods such as |
For a project that is using MATLAB POUNDERS presently, my
callpounders.m
sets the current working directory (CWD) to the location in which needed configuration files are stored and where I want results to be written. This is completely unrelated to where IBCDFO is installed.However, this doesn't presently work because of the relative path in
IBCDFO/pounders/m/pounders.m
Lines 7 to 9 in 0b6d017
which I believe looks for
general_h_funs
with respect to the CWD.@jmlarson1 @mmenickelly @wildsm Can we assume that if someone is calling a specific
pounders.m
, then they should be using thegeneral_h_funs
in the same clone? If so, then how about usingThe text was updated successfully, but these errors were encountered: