Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.x] [TSVB] Replace deprecated moving_avg by moving_fn aggregation (#36624) #39275

Merged
merged 1 commit into from
Jun 20, 2019
Merged

[7.x] [TSVB] Replace deprecated moving_avg by moving_fn aggregation (#36624) #39275

merged 1 commit into from
Jun 20, 2019

Conversation

gospodarsky
Copy link

Backports the following commits to 7.x:

@elasticmachine
Copy link
Contributor

💔 Build Failed

@gospodarsky
Copy link
Author

gospodarsky commented Jun 20, 2019

Jenkins, test this - CI outage

@gospodarsky
Copy link
Author

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

* Take model options away in a separate file

* Connect user scripting to models

* Add inputs to the panel for managing alpha, beta, gamma params

* Cover model types and scripts by tests

* Change default export of the bucket transform to the permanent one and write tests for moving_average

* Add a migration script from mov_avg to mov_fn

* Remove redundant translations

* Remove old tests

* Fix issues of PR

* Revert yarn.lock

* Fix issues regarding localization

* Remove extra license

* Remove redundant translations

* Move the replaceMovAvgToMovFn hook to 7.3.0

* Fix reviews

* Add a migration test

* Set proper default values and turn hint on for holt-winter only

* Format touched files by Prettier

* Import settings from moving_avg

* Wrap changes to the try/catch statement and log exceptions
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@gospodarsky gospodarsky merged commit 319341b into elastic:7.x Jun 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants