Skip to content

[ML] Fix possible cause for "Bad variance scale nan" log errors #1225

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

Merged
merged 3 commits into from
May 11, 2020

Conversation

tveasey
Copy link
Contributor

@tveasey tveasey commented May 11, 2020

Following on from #1114, there are a couple of places where we have code which could in theory generate 0/0. These would be a candidate root cause for #364 along with #1157.

This should generally have no impact on results until we'd actually hit the problem case.

@tveasey tveasey changed the title [ML] Trap possible divide by zero [ML] Fix possible cause for "Bad variance scale nan" log errors May 11, 2020
Copy link
Contributor

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, although I suggested adding a comment that might prevent someone accidentally removing one of the fixes in the future.

Also, if the effect of this is bad then maybe the fix should go in 7.7.1 as well as 7.8.0?

@tveasey tveasey added the v7.7.1 label May 11, 2020
@tveasey tveasey merged commit ecfe81f into elastic:master May 11, 2020
@tveasey tveasey deleted the numeric-stability branch May 11, 2020 15:17
tveasey added a commit to tveasey/ml-cpp-1 that referenced this pull request May 11, 2020
tveasey added a commit to tveasey/ml-cpp-1 that referenced this pull request May 11, 2020
tveasey added a commit to tveasey/ml-cpp-1 that referenced this pull request May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants