Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 39d7013

Browse files
authoredJun 5, 2024
fix: ARIMAPlus loads auto_arima_min_order param (#752)
1 parent 7899749 commit 39d7013

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed
 

‎tests/system/large/ml/test_forecasting.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,7 @@ def test_arima_plus_model_fit_params(time_series_df_default_index, dataset_id):
127127
assert reloaded_model.horizon == 100
128128
assert reloaded_model.auto_arima is True
129129
assert reloaded_model.auto_arima_max_order == 4
130-
# TODO(garrettwu): now BQML doesn't populate auto_arima_min_order
131-
# assert reloaded_model.auto_arima_min_order == 1
130+
assert reloaded_model.auto_arima_min_order == 1
132131
assert reloaded_model.data_frequency == "DAILY"
133132
assert reloaded_model.holiday_region == "US"
134133
assert reloaded_model.clean_spikes_and_dips is False

0 commit comments

Comments
 (0)
Failed to load comments.