Opened 9 years ago
Closed 8 years ago
#2529 closed defect (done)
AR1 is not evaluated correctly on the test partition
Reported by: | gkronber | Owned by: | gkronber |
---|---|---|---|
Priority: | medium | Milestone: | HeuristicLab 3.3.15 |
Component: | Algorithms.DataAnalysis | Version: | 3.3.13 |
Keywords: | Cc: |
Description
The AR1 model should regress on the previous model outputs in the test set (therefore the name auto-regressive).
Currently the AR1 model uses the actual value of the target variable at t-1 even on the test partition.
This is also true for the symbol "auto-regressive variable" for symbolic time-series modeling.
Attachments (1)
Change History (17)
Changed 9 years ago by gkronber
comment:1 Changed 9 years ago by gkronber
comment:2 Changed 9 years ago by gkronber
Attached file shows unwanted behavior.
comment:3 Changed 9 years ago by gkronber
- Milestone changed from HeuristicLab 3.3.14 to HeuristicLab 3.3.x Backlog
comment:4 Changed 8 years ago by gkronber
- Status changed from new to accepted
comment:5 Changed 8 years ago by gkronber
The caching of target variable values in the interpreter for time series models has a bug (the values are not initialized correctly e.g. when the problem changes).
A specific version of the line chart needs to be implemented which uses the complete test partition as prognosis horizon.
comment:6 Changed 8 years ago by gkronber
r14422: added a way to calculate prognosed values for the whole test partition and added a specific implementation of the line chart for time series models
comment:7 Changed 8 years ago by gkronber
r14423: removed caching of traget variable values because it didn't work correctly.
comment:8 follow-up: ↓ 9 Changed 8 years ago by gkronber
TODO: check if sample still produces the same values.
comment:9 in reply to: ↑ 8 Changed 8 years ago by gkronber
Replying to gkronber:
TODO: check if sample still produces the same values.
Nope...
Stacktraces: (new) MSTest: HeuristicLab.Tests.GPTimeSeriesSampleTest.RunGpTimeSeriesSampleTest Assert.AreEqual failed. Expected a difference no greater than <1E-08> between expected value <0,0209527534151996> and actual value <0,0490851910414951>.
comment:10 Changed 8 years ago by gkronber
Merge with #2718
comment:11 Changed 8 years ago by gkronber
- Milestone changed from HeuristicLab 3.3.x Backlog to HeuristicLab 3.3.15
comment:12 Changed 8 years ago by gkronber
r14529: updated target result values in unit test (previously the interpreter was buggy)
comment:13 Changed 8 years ago by gkronber
- Owner changed from gkronber to mkommend
- Status changed from accepted to reviewing
comment:14 Changed 8 years ago by mkommend
- Owner changed from mkommend to gkronber
- Status changed from reviewing to readytorelease
OK. I still find it slightly misleading that the training and test range behave differently (estimation vs. prognosis).
comment:15 Changed 8 years ago by gkronber
comment:16 Changed 8 years ago by gkronber
- Resolution set to done
- Status changed from readytorelease to closed
Problematic code in AutoregressiveModeling.cs:
The symbol AutoregressiveTargetVariable is treated in the same way as any other lagged variable in the interpreter