-
Notifications
You must be signed in to change notification settings - Fork 910
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
Add --conf-source
option to kedro ipython
#2920
Comments
--conf_source
option to kedro ipython
--conf-source
option to kedro ipython
Hi @ankatiyar, I would need some help to tackle this issue. I read the PR #2117, but I am unsure how this should be implemented for Is it that it should be first added as an kedro/kedro/ipython/__init__.py Line 62 in e1823ed
Then in function kedro/kedro/ipython/__init__.py Line 81 in e1823ed
And should be used as an input to load
kedro/kedro/ipython/__init__.py Line 93 in e1823ed
Please let me know what your ideas are.. 💡 |
@laizaparizotto I think what you've suggested looks about right, might be some more things are required. I haven't looked into the implementation myself but I can and in the meantime, if you can try this out too and open a PR(or a draft PR), we could collaborate on the implementation. |
Sure, @ankatiyar. I will make the changes and open a PR soon. Thank you for the help |
Description
Right now we can pass
env
andparams
as options withkedro ipython
/%reload_kedro
. Add--conf-source
option as well.Context
User question from slack :
Possible Implementation
Changes to be made in
kedro/ipython/__init__.py
PR for when
--conf-source
was added tokedro run
- #2117The text was updated successfully, but these errors were encountered: