* Pass ... from dyplot.prophet to dygraph to be able to set additional parameters
* Add documentation for ... in dyplot.prophet
Co-authored-by: Ben Letham <bletham@gmail.com>
Ensures that `add_regressor` and `add_seasonality` are valid column names
to R, to ensure that the generated columns are then used downstream to fit
the model.
Why not put it in `validate_column_names`? Because `validate_column_names` is
also used to validate if holiday names (which can be scalar values in columns) are
valid. We want to allow `c('seans-bday', 'Xmas')` as a valid holiday input, so we
cannot then put it there.
Tested these changes by using devtools::testthat().
Resolves: #996
In reverse order of importance, this change addresses:
- #980, by adding validation to ensure that `fourier.order` > 0
- changes all the tests that `expect_error` using the argument
`fourier_order`, as the argument is named `fourier.order`, so
the intent of the test is to validate the holiday names rather
than to check the use of the correct argument name
Resolves: #980
`dplyr::as_data_frame()` is really `tibble::as_tibble()`. An upcoming
release of tibble will be stricter about column names and, by default,
you can't have duplicate column names. This change allows prophet to
work with current CRAN tibble (v1.4.2) and the coming release.
cc @krlmlr
* Allow both both hoidays and append holidays
Match holidays in predict and fit
Add test for append_holiday features; minor fixes
Add column name validation for append_holidays names; allow only one country
Fix bug about holiday year different from input year and also other bugs
Change function description
Add append holiday feature for R
* Add test for R/Pyrhon; fix bugs
