-
Notifications
You must be signed in to change notification settings - Fork 928
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
Improve runtime params documentation #3562
Comments
We aren't mentioning [A separate issue, I think, is that probably To help engineering determine if there's anything they need to change in terms of content, a couple of queries:
These aren't questions necessarily for you @Galileo-Galilei but you're welcome to supply any thoughts. @merelcht will work with the team to prioritise any changes needed. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
We agreed to add an admonition from the bottom of https://docs.kedro.org/en/latest/configuration/parameters.html#how-to-specify-parameters-at-runtime linking to https://docs.kedro.org/en/latest/configuration/advanced_configuration.html#how-to-override-configuration-with-runtime-parameters-with-the-omegaconfigloader, mentioning |
Description
The documention about
runtime_params
does not even mention the resolver and how to configure the catalog:This is very confusing and not very informative.
Documentation page (if applicable)
https://docs.kedro.org/en/latest/configuration/parameters.html
Context
When looking for "kedro runtime_params" on google, above page is the first result and it is not where we want users to be redirected (not sure why because this page also exists and is the right one : https://docs.kedro.org/en/latest/configuration/advanced_configuration.html). We should at least redirect to the correct page.
Implementation
Copying @astrojuanlu's comment here to make the ask of this ticket clear -
The text was updated successfully, but these errors were encountered: