Merge pull request #297 from jmfee-usgs/residual-refactor
Refactor for readability - Looks good. The clean-up is making it a little easier to interpret the algorithm's logic, and TBH, I'm have some small doubts about the mathematical correctness of some things. But I'm pretty sure it faithfully captures the original spreadsheet logic, which is all we can do for now.
No related branches found
No related tags found
Showing
- Pipfile 3 additions, 1 deletionPipfile
- Pipfile.lock 90 additions, 73 deletionsPipfile.lock
- geomagio/TimeseriesUtility.py 27 additions, 0 deletionsgeomagio/TimeseriesUtility.py
- geomagio/residual/Calculation.py 227 additions, 352 deletionsgeomagio/residual/Calculation.py
- geomagio/residual/Measurement.py 78 additions, 3 deletionsgeomagio/residual/Measurement.py
- geomagio/residual/MeasurementType.py 39 additions, 12 deletionsgeomagio/residual/MeasurementType.py
- geomagio/residual/Ordinate.py 0 additions, 22 deletionsgeomagio/residual/Ordinate.py
- geomagio/residual/Reading.py 67 additions, 31 deletionsgeomagio/residual/Reading.py
- geomagio/residual/SpreadsheetAbsolutesFactory.py 38 additions, 48 deletionsgeomagio/residual/SpreadsheetAbsolutesFactory.py
- geomagio/residual/WebAbsolutesFactory.py 4 additions, 0 deletionsgeomagio/residual/WebAbsolutesFactory.py
- geomagio/residual/__init__.py 25 additions, 4 deletionsgeomagio/residual/__init__.py
- test/TimeseriesUtility_test.py 42 additions, 0 deletionstest/TimeseriesUtility_test.py
- test/residual_test/residual_test.py 44 additions, 55 deletionstest/residual_test/residual_test.py
Loading
Please register or sign in to comment