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

[UI] Unable to select Experiment parameters #1620

Closed
andreyvelich opened this issue Aug 13, 2021 · 3 comments · Fixed by #1652
Closed

[UI] Unable to select Experiment parameters #1620

andreyvelich opened this issue Aug 13, 2021 · 3 comments · Fixed by #1652

Comments

@andreyvelich
Copy link
Member

/kind bug

I built Katib UI image from the latest master commit.
I am getting the following error:
Experiment Bug

/assign @kimwnasptd
/priority p0

@andreyvelich
Copy link
Member Author

@kimwnasptd I was able to find this problem.
That happens if my K8s cluster doesn't have Trial Templates.
Once I deployed this ConfigMap with the appropriate Label, form was complete.

I think here: https://github.com/kubeflow/katib/blob/master/pkg/new-ui/v1beta1/frontend/src/app/pages/experiment-creation/trial-template/trial-template.component.html#L81-L88, before re-calculate Trial parameters, we have to make sure that Templates were received properly.

Also, when user selects ConfigMaps, we should display that templates don't exists.
We did it in the previous UI: https://github.com/kubeflow/katib/blob/master/pkg/ui/v1beta1/frontend/src/components/Common/Create/Params/Trial/TrialConfigMap.jsx#L224.

@kimwnasptd
Copy link
Member

Thanks for finding and reporiting this @andreyvelich. Both your suggestions make sense.

I'll be on PTO on this week but I'll start working immediately on this once I'll be back.

@kimwnasptd
Copy link
Member

I'm back, picking this up to include this fix in the 1.4 Kubeflow release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants