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

[ML] adds feature_importance_baseline object to model metadata #63172

Conversation

benwtrent
Copy link
Member

this adds the new field feature_importance_baseline and allows it to be optionally be included in the model's metadata.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml)

@benwtrent benwtrent requested a review from valeriy42 October 2, 2020 11:15
@benwtrent
Copy link
Member Author

@elasticmachine update branch

@benwtrent
Copy link
Member Author

run elasticsearch-ci/2

Copy link
Member

@davidkyle davidkyle left a comment

Choose a reason for hiding this comment

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

LGTM

private static final String NAME = "feature_importance_class_baseline";

public static final ParseField CLASS_NAME = new ParseField("class_name");
public static final ParseField BASELINE = new ParseField("baseline");
Copy link
Member

Choose a reason for hiding this comment

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

nit: you could reuse the BASELINE parsefield defined in the parent class

@benwtrent benwtrent merged commit 0c56516 into elastic:master Oct 5, 2020
@benwtrent benwtrent deleted the feature/ml-analytics-feature-importance-baseline branch October 5, 2020 12:14
benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Oct 5, 2020
…ic#63172)

this adds the new field `feature_importance_baseline` and allows it to be optionally be included in the model's metadata.

Related to: elastic/ml-cpp#1522
benwtrent added a commit that referenced this pull request Oct 5, 2020
… (#63237)

this adds the new field `feature_importance_baseline` and allows it to be optionally be included in the model's metadata.

Related to: elastic/ml-cpp#1522
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.

4 participants