Skip to content

Commit

Permalink
rename explain api endpoint to general explainDataFrameAnalytics
Browse files Browse the repository at this point in the history
  • Loading branch information
alvarezmelissa87 committed Jan 6, 2020
1 parent 09f63f4 commit 5cfd1c9
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ export const CreateAnalyticsForm: FC<CreateAnalyticsFormProps> = ({ actions, sta
const jobConfig = getJobConfigFromFormState(form);
delete jobConfig.dest;
delete jobConfig.model_memory_limit;
const resp = await ml.dataFrameAnalytics.estimateDataFrameAnalyticsMemoryUsage(jobConfig);
const resp = await ml.dataFrameAnalytics.explainDataFrameAnalytics(jobConfig);
setFormState({
modelMemoryLimit: resp.memory_estimation?.expected_memory_without_disk,
});
Expand All @@ -192,7 +192,9 @@ export const CreateAnalyticsForm: FC<CreateAnalyticsFormProps> = ({ actions, sta
const loadFieldOptions = async () => {
setFormState({
loadingFieldOptions: true,
// clear when the source index changes
dependentVariable: '',
excludes: [],
// Reset outlier detection sourceIndex checks to default values if we've switched to regression
sourceIndexFieldsCheckFailed: false,
sourceIndexContainsNumericalFields: true,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ export const dataFrameAnalytics = {
data: evaluateConfig,
});
},
estimateDataFrameAnalyticsMemoryUsage(jobConfig) {
explainDataFrameAnalytics(jobConfig) {
return http({
url: `${basePath}/data_frame/analytics/_explain`,
method: 'POST',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,9 +89,7 @@ declare interface Ml {
getDataFrameAnalyticsStats(analyticsId?: string): Promise<GetDataFrameAnalyticsStatsResponse>;
createDataFrameAnalytics(analyticsId: string, analyticsConfig: any): Promise<any>;
evaluateDataFrameAnalytics(evaluateConfig: any): Promise<any>;
estimateDataFrameAnalyticsMemoryUsage(
jobConfig: DeepPartial<DataFrameAnalyticsConfig>
): Promise<any>;
explainDataFrameAnalytics(jobConfig: DeepPartial<DataFrameAnalyticsConfig>): Promise<any>;
deleteDataFrameAnalytics(analyticsId: string): Promise<any>;
startDataFrameAnalytics(analyticsId: string): Promise<any>;
stopDataFrameAnalytics(
Expand Down
2 changes: 1 addition & 1 deletion x-pack/legacy/plugins/ml/server/client/elasticsearch_ml.js
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ export const elasticsearchJsPlugin = (Client, config, components) => {
method: 'POST',
});

ml.estimateDataFrameAnalyticsMemoryUsage = ca({
ml.explainDataFrameAnalytics = ca({
urls: [
{
fmt: '/_ml/data_frame/analytics/_explain',
Expand Down

0 comments on commit 5cfd1c9

Please sign in to comment.