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

[FEATURE] Add missing Integration test for PPL integration with MLCommons #1563

Open
dhrubo-os opened this issue Apr 19, 2023 · 0 comments
Open
Labels
infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc.

Comments

@dhrubo-os
Copy link
Contributor

Is your feature request related to a problem?
Right now PPL integration with MLCommons doesn't have any corresponding integration test.

In 2.7, we found out that the following query was breaking to execute:

POST _plugins/_ppl
{
  "query": "source=iris_data | fields petal_length_in_cm,petal_width_in_cm | kmeans centroids=3"
}

Details in this PR

This error should caught up during integration testing.

What solution would you like?
Need to write corresponding Integration test.

@dhrubo-os dhrubo-os added enhancement New feature or request untriaged labels Apr 19, 2023
@dai-chen dai-chen added infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc. and removed enhancement New feature or request untriaged labels Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc.
Projects
None yet
Development

No branches or pull requests

2 participants