Nok Lam Chan

Results 574 comments of Nok Lam Chan
trafficstars

I like the **keyword-only** idea and it should make things more flexible, if we shuffle the order in the future it won't be a breaking change anymore, is it correct?...

@merelcht I have been asking about this so I am all for itπŸ˜„

I don't think mutating DataCatalog is a good thing to do. I believe the hook system was designed in a way to avoid this exactly (correct me if I am...

> That's fair enough, but not the impression that I got when I saw a DataCatalog.add method existed πŸ€” How could we reduce confusion here? I think we need to...

Maybe the action item for this ticket is: - Create documentation to explain `DataCatalog` immutability Is there a good reason Kedro should start supporting this?

> That would actually make this really customizable with sensible default. What do you mean? The proposal does not requires using ConfigLoader, it only use the OmegaConf cli parse feature....

A quick attempt as I come across some question in Slack today. Originally I want to take regex approach https://stackoverflow.com/questions/26633452/how-to-split-by-commas-that-are-not-within-parentheses but without success. https://github.com/kedro-org/kedro/pull/3327 - this is not ready to...

It was never put into a sprint, I made a draft PR to see what it takes to add just list as an example. It requires building a parser basically,...

@stichbury I am glad you like the idea, I find this initially from https://twitter.com/_ScottCondron/status/1363494433715552259?t=YOucme8E9_m8GDbVBYCG4g&s=19 I would like to see it to be shared on platform/social media. We talked about the...

Moving this out from `inbox` since I don't think we have capacity for this in short term.