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][ML] Relax assertion to expect at least on trained model (#67710) #67758

Conversation

dimitris-athanasiou
Copy link
Contributor

... for data frame analytics native tests. This fixes assertion
failures seen in #67581.

This commit also enables debug logging for RegressionIT and
unmutes test_stop_and_restart in order to collect more information
about why some times progress gets stuck after loading_data.

Backport of #67710

…c#67710)

... for data frame analytics native tests. This fixes assertion
failures seen in elastic#67581.

This commit also enables debug logging for `RegressionIT` and
unmutes `test_stop_and_restart` in order to collect more information
about why some times progress gets stuck after `loading_data`.

Backport of elastic#67710
@dimitris-athanasiou dimitris-athanasiou added >test Issues or PRs that are addressing/adding tests :ml Machine learning backport labels Jan 20, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml)

@dimitris-athanasiou dimitris-athanasiou merged commit 16caeae into elastic:7.x Jan 20, 2021
@dimitris-athanasiou dimitris-athanasiou deleted the relax-assertion-to-expect-at-least-one-trained-model-7x branch January 20, 2021 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :ml Machine learning >test Issues or PRs that are addressing/adding tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants