ykang / gratis

GRATIS: GeneRAting TIme Series with diverse and controllable characteristics
https://github.com/ykang/gratis
GNU General Public License v3.0
76 stars 29 forks source link

Failure to simulate `seas_pacf` feature.... #10

Closed michael135 closed 5 years ago

michael135 commented 5 years ago

Hello, I had en error, when I tried to simulate seas_pacf feature:

x <- generate_ts_with_target(n = 2, ts.length = 60, freq = 1, seasonal = 0,
                             features = c('pacf_features'),
                             selected.features = c('seas_pacf'),
                             target = c(.01))

The output is:

Error in { : task 1 failed - "Unknown column `seas_pacf`
Backtrace:

What was done wrong?

michael135 commented 5 years ago

If we will change freq = 2 it will run without en error.

ykang commented 5 years ago

Yes because seas_pacf is a feature for seasonal time series. If freq = 1, seas_pacf is not in the candidate set of features.

michael135 commented 5 years ago

thank you @ykang , I would recommend to add it at function documentation. For example at docstring of https://github.com/ykang/tsgeneration/blob/master/R/generate_ts_with_target.R

Currently, it's unclear from documentation, which value of freq are relevant for what.

ykang commented 5 years ago

Thanks Michael for your suggestion.

Sent from my iPhone

On Apr 22, 2019, at 13:31, Michael Dym notifications@github.com<mailto:notifications@github.com> wrote:

thank you @ykanghttps://github.com/ykang , I would recommend to add it at function documentation. For example at docstring of https://github.com/ykang/tsgeneration/blob/master/R/generate_ts_with_target.R

Currently, it's unclear from documentation, which value of freq are relevant for what.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ykang/tsgeneration/issues/10#issuecomment-485329695, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAJFJVRQE2OUEUI7CKM3LE3PRVESXANCNFSM4HG3VG5A.