Transport-for-the-North / caf.distribute

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

Sectoral constraint #36

Open isaac-tfn opened 6 months ago

isaac-tfn commented 6 months ago

Changes

A new furnessing method which allows constraint to a more aggregate matrix. This is envisaged as being used for disaggregating matrices, where the original trip ends are disagreggated using caf.space/caf.toolkit, seed values generated using original cost function params, and the matrix being controlled back to the original, more aggregate matrix during furnessing.

It is designed to work where the new, disaggregate, zone system nests perfectly within the original zone system; i.e. translation factors from disaggregate to aggregate are all 1.

Task Checklist

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 54.09836% with 28 lines in your changes missing coverage. Please review.

Project coverage is 78.84%. Comparing base (a03bda2) to head (b4001fc). Report is 23 commits behind head on main.

:exclamation: Current head b4001fc differs from pull request most recent head 82a02c6

Please upload reports for the commit 82a02c6 to get more accurate results.

Files Patch % Lines
src/caf/distribute/furness.py 54.54% 25 Missing :warning:
src/caf/distribute/gravity_model/multi_area.py 50.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #36 +/- ## ========================================== - Coverage 81.38% 78.84% -2.55% ========================================== Files 9 9 Lines 634 690 +56 ========================================== + Hits 516 544 +28 - Misses 118 146 +28 ```

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