Transport-for-the-North / caf.distribute

https://cafdistribute.readthedocs.io/en/stable/
Other
0 stars 1 forks source link

CAFDI-7 - Create Multi TLD Gravity Model MVP #7

Closed wsp-mbuckley closed 9 months ago

wsp-mbuckley commented 1 year ago

Describe Changes

Add class to calibrate gravity model to multiple cost distributions.

Task Checklist

BenTaylor-TfN commented 1 year ago

Finished #6 now. Also updated the packaging system in #8.

Rebased this off of the new main to keep the commit messages in here a bit clearer. No significant changes that you weren't already aware of.

codecov[bot] commented 1 year ago

Codecov Report

Attention: 89 lines in your changes are missing coverage. Please review.

Comparison is base (30e4c8c) 85.09% compared to head (e9672b2) 81.38%.

Files Patch % Lines
src/caf/distribute/gravity_model/multi_area.py 81.63% 36 Missing :warning:
src/caf/distribute/gravity_model/core.py 26.66% 22 Missing :warning:
src/caf/distribute/gravity_model/single_area.py 82.75% 20 Missing :warning:
src/caf/distribute/utils.py 52.17% 11 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7 +/- ## ========================================== - Coverage 85.09% 81.38% -3.71% ========================================== Files 7 9 +2 Lines 416 634 +218 ========================================== + Hits 354 516 +162 - Misses 62 118 +56 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

isaac-tfn commented 9 months ago

@BenTaylor-TfN I think I've made those changes now