Skip to content

Commit ce6e71b

Browse files
committed
Review comments
1 parent e3e8db8 commit ce6e71b

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

docs/CHANGELOG.asciidoc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@ Improve and use periodic boundary condition for seasonal component modeling ({pu
3131

3232
=== Bug Fixes
3333

34+
Age seasonal components in proportion to the fraction of values with which they're updated ({pull}88[#88])
35+
3436
=== Regressions
3537

3638
=== Known Issues

lib/maths/CSeasonalComponentAdaptiveBucketing.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -240,8 +240,8 @@ void CSeasonalComponentAdaptiveBucketing::propagateForwardsByTime(double time, b
240240
if (time < 0.0) {
241241
LOG_ERROR(<< "Can't propagate bucketing backwards in time");
242242
} else if (this->initialized()) {
243-
time *= m_Time->fractionInWindow();
244-
double factor{std::exp(-this->CAdaptiveBucketing::decayRate() * time)};
243+
double factor{std::exp(-this->CAdaptiveBucketing::decayRate() *
244+
m_Time->fractionInWindow() * time)};
245245
this->CAdaptiveBucketing::age(factor);
246246
for (auto& bucket : m_Buckets) {
247247
bucket.s_Regression.age(factor, meanRevert);

0 commit comments

Comments
 (0)