Currently a lot of global state is used (inputs, outputs etc). This makes it very hard to test the application and generally requires running scripts via subprocess which is hard to debug.
This removes the global state in preference for a PriorConfig object to carry around the options. This config object is still created from environment variables.
This also enables the GeoJSON test which was previously disabled because of dependency issues.
Checklist
Please confirm that this pull request has done the following:
[x] Tests added
[x] Documentation added (where applicable)
Notes
This also moves the input and output directories into the data/ directory
lewisjared
commented
3 months ago
Description
Currently a lot of global state is used (inputs, outputs etc). This makes it very hard to test the application and generally requires running scripts via
subprocesswhich is hard to debug.This removes the global state in preference for a
PriorConfigobject to carry around the options. This config object is still created from environment variables.This also enables the GeoJSON test which was previously disabled because of dependency issues.
Checklist
Please confirm that this pull request has done the following:
Notes
This also moves the input and output directories into the
data/directory