Dmitry Sorokin
Dmitry Sorokin
Thanks for the investigation and PR, @ElenaKhaustova! I agree with @noklam that relying solely on lambda functions for lazy saving doesn't seem like a generic solution. While it is a...
> Based on the above arguments, my suggestion would be to make lazy saving a default behaviour like it's done for lazy loading now. For that, we can wrap and...
Thanks, @ElenaKhaustova! If I understand correctly, the default behavior will remain the same as the current one. However, we are adding the option to use lazy=False, which will prevent Callables...
Hi @MarcelBeining. Thanks for raising this! I think it makes a lot of sense. Would you be interested in submitting a pull request for this? If not, the Kedro maintainers...
Thank you very much for the comprehensive review, @merelcht! I agree that the current flow is not ideal. Let's brainstorm how we can restructure it. My main idea was to...
@merelcht, @ankatiyar, thank you for your reviews! I have addressed most of your comments. I would appreciate any advice on micropackaging and providing a clearer explanation of namespaces. Additionally, I...
@astrojuanlu, @noklam, thank you for your comments. I have merged modular pipelines with pipeline objects on one page to simplify things for beginner users. I felt that it wasn't essential...
I have changed the structure according to @astrojuanlu's proposal and addressed other minor comments. I am now re-requesting reviews.
> Also wondering why the images are smaller now? 😅 I resized them to smaller ones because previously I saw them (on the screen below) in a very weird way:
> Gave another pass, left some comments. We're on the right path! Thank you for your additional comments, @astrojuanlu. I agree with them and have modified the docs accordingly. It...