jessegrabowski / tsdisagg

Tools for converting low time series data to high frequency
MIT License
4 stars 2 forks source link

Fix C matrix #10

Closed jessegrabowski closed 2 months ago

jessegrabowski commented 2 months ago

Closes #6

The C matrix was previously built under an assumption that the high and low frequency data were perfectly aligned. This PR relaxes this by following the preprocessing used in the R package

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 98.82353% with 1 line in your changes missing coverage. Please review.

Project coverage is 86.41%. Comparing base (d1ff30e) to head (2aafc88). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
tsdisagg/ts_disagg.py 98.82% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10 +/- ## ========================================== + Coverage 82.07% 86.41% +4.34% ========================================== Files 3 3 Lines 357 368 +11 ========================================== + Hits 293 318 +25 + Misses 64 50 -14 ```

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