Merel Theisen

Results 327 comments of Merel Theisen

The exists() method is not an abstract method and so doesn't have to be implemented. The original `SharedMemoryDataset` was wrapped around `MemoryDataset` directly so I guess that's why the warning...

Reported again recently: https://github.com/kedro-org/kedro/issues/4060 Updated the scope to double check where `exists` is called and if it makes sense to implement this method on `SharedMemoryDataset`. Also related: https://github.com/kedro-org/kedro/issues/4078

Hi @Lasica, are you still interested in creating a PR for this?

Since there's no consensus on this POC and it's a breaking change, I suggest to close this for now. We have several tickets open regarding performance issues and the `DataCatalog`...

> QQ also about the `vale` usage in CI. I noticed that it produces some warnings on this PR, but apparently on many other existing MD files. So it's like...

Hi @stichbury, we've discussed this in backlog grooming and decided we'd like to get it in as docs. I really appreciate any help you can give getting this in shape...

While this doesn't entirely close #3418, I do think it's a valuable addition to our docs and I would very much like this to go in. @noklam Could you have...

Hi @JenspederM, thanks for flagging this issue. Can I ask what your use case is for printing the result of `find_pipelines()`? This method has been added to enable auto discovery...

Hi @tzdanowicz, thank you for creating this issue. This sounds like a bug on our end, so I'll make sure we address it and put it on our backlog.

@tzdanowicz this has been fixed in #3950 and will be released in our next release `0.19.7`