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

[DOCS] Adding ML-specific prerequisites to setup docs #42529

Merged
merged 1 commit into from
May 24, 2019

Conversation

droberts195
Copy link
Contributor

  1. On Windows the Microsoft UCRT library is required
  2. SSE4.2 instructions must be supported by the CPU

Closes #42295

1. On Windows the Microsoft UCRT library is required
2. SSE4.2 instructions must be supported by the CPU

Closes elastic#42295
@droberts195 droberts195 requested a review from lcawl May 24, 2019 15:11
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@lcawl
Copy link
Contributor

lcawl commented May 24, 2019

Should we perhaps also list these in the machine learning limitations?: https://www.elastic.co/guide/en/elastic-stack-overview/master/ml-limitations.html
If so, I can create a corresponding PR there.

@droberts195
Copy link
Contributor Author

Should we perhaps also list these in the machine learning limitations?

Yes, certainly the SSE4.2 one should go there. I’m not so sure the UCRT one is a “limitation” though. More of a prerequisite.

Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM and builds successfully

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.

[ML] Document ML C++ specific pre-requisites
4 participants