-
Notifications
You must be signed in to change notification settings - Fork 112
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
Allow users to see parameter configs from past runs in experiment tracking #1020
Comments
There needs to be a mechanism for the user re-run with those same commands. Either generating the CLI command or in the UI |
To do on the Framework side:
|
Thought: what happens if there's a folder of multiple |
@tynandebold Kedro (within the same |
Further user demand for feature |
Closing this. We'll reopen this ticket if we look at this again downstream. |
Description
It seems suboptimal we don't give users a way to view their parameter configurations from Kedro runs in experiment tracking. They could conceivably do this via GitHub if they're continually committing their changes, though if done that way it's more arduous and isn't done via Viz. And if they don't do that, then they'll lose any ability to see these values.
Idea initially conceived and discussed with @samhiscoxqb and @marianansilva.
Problem statement
As a user, I want to be able to see in experiment tracking what my parameter value was for a run sometime in the past, especially when the parameter wasn’t committed to git.
Context
I propose something like the following, where we add a Run Config metadata field:
Clicking on that field would open a modal, where the user would be able to view their parameters for that run (more nicely formatted in the future with syntax highlighting):
Possible Implementation
Kedro would have to save a (versioned?) parameters file somewhere at the time of performing a
kedro run
. The contents of that file would then need to be read and passed via therun_metadata
query to the frontend, where we display it.Possible Alternatives
Not so much an alternative, and instead an idea for a future version:
Checklist
The text was updated successfully, but these errors were encountered: