Skip to content

Commit

Permalink
Project Params Hotfix (G-Research#62)
Browse files Browse the repository at this point in the history
* Hotfix: Add support for experimentNames in getProjectParams

* Fetch project params on selectedExperiment change
  • Loading branch information
jescalada authored and vinayan3 committed Aug 28, 2024
1 parent 7ba8118 commit 051d41e
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 6 deletions.
13 changes: 8 additions & 5 deletions src/src/services/api/projects/projectsService.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,17 @@ function fetchActivityData(): IApiRequest<any> {

function getProjectParams(
sequences: string[] = ['metric'],
selectedExperimentNames: string[] = [],
): IApiRequest<IProjectParamsMetrics> {
const query = sequences.reduce(
(acc: string, sequence: string, index: number) => {
const query =
sequences.reduce((acc: string, sequence: string, index: number) => {
acc += `${index === 0 ? '?' : '&'}sequence=${sequence}`;
return acc;
},
'',
);
}, '') +
selectedExperimentNames.reduce((acc: string, experimentName: string) => {
acc += `&experiment_names=${experimentName}`;
return acc;
}, '');
return API.get<IProjectParamsMetrics>(endpoints.GET_PROJECTS_PARAMS + query);
}

Expand Down
4 changes: 3 additions & 1 deletion src/src/services/models/explorer/metricsModelMethods.ts
Original file line number Diff line number Diff line change
Expand Up @@ -223,8 +223,10 @@ function getMetricsAppModelMethods(
}

function fetchProjectParamsAndUpdateState() {
const selectedExperimentNames =
model.getState()?.config?.select.selectedExperimentNames;
projectsService
.getProjectParams(['metric'])
.getProjectParams(['metric'], selectedExperimentNames)
.call()
.then((data) => {
const advancedSuggestions: Record<any, any> = getAdvancedSuggestion(
Expand Down

0 comments on commit 051d41e

Please sign in to comment.