-
Notifications
You must be signed in to change notification settings - Fork 459
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
Persist meta data of experiments #1128
Comments
Issue-Label Bot is automatically applying the labels:
Please mark this comment with 👍 or 👎 to give our bot feedback! |
@sperlingxx Thank you for the issue. SGTM. Maybe we can control it with resuming Experiment flag. |
@andreyvelich I fully agree with the approach "control it with resuming Experiment flag". Shall we add this issue into 2020 roadmap? |
@gaocegege @johnugeorge Any thoughts? |
related:#1150 (comment) |
Resuming Experiment was implemented, feed free to re-open the issue if you have any other questions. |
I have one question about this issue. We can now control whether the Suggestion deploy is deleted after the experiment is finished by setting However, Trial, Suggestion, and Experiment CRs still remain. As discussed at the beginning of this issue, how about adding an option to persist all meta data of the experiment in DB and delete Katib CRs? |
/kind feature
Describe the solution you'd like
Meta data generated during HPO process is worthy to restore. It can be used to guide subsequent tuning works. Maybe it can also be precious data of further academic research.
To me, since katib has DB abstraction layer (KatibDBInterface) , we can extend it to persist meta data such as experiment settings and corresponding trial information (param assignments and metrics).
Anything else you would like to add:
In current katib UI system, visualization will be unavailable once experiment instance deleted. If we can persist all information in DB, it is possible for users to review all history experiments. I think it will be helpful.
/cc @andreyvelich @gaocegege @richardsliu @hougangliu @johnugeorge
The text was updated successfully, but these errors were encountered: