Add environment variable for output directory #54
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR:
QUANDARY_BASE_DATADIR
to quandary.py to set a base output directory for both the config.cfg and output files. A user can still pass in adatadir
and (if this is a relative path) this will be evaluated relative to theQUANDARY_BASE_DATADIR
or if that is unset, then it will use the current working directory. When the environment variable is unset the behavior should be the same as it is currently. As the python code changes directories before executing quandary, it was not necessary to make changes to the C++ code for this.tests/python
. These are also regression tests but need to be in their own folder, since the other regression tests rely on the folder structure for test cases.regression
andperformance
test category to pytest to offer an extra way to run the different test subsets in addition to using the folders.pytest -m regression
will run both thetests/python
andtests/regression
. Update docs to explain this.