https://kedro.org/ logo
#user-research
Title
# user-research
n

Nok Lam Chan

08/09/2023, 8:26 PM
If Kedro supports unpacking parameters dictionary, would it causes more confusion than convinence? IMO it is most beneficial if the function is reused multiple time and reduce the need of a thin function wrapper (and an extra file maybe). I am particular interested with people who use kedro to develop packages. https://github.com/kedro-org/kedro/issues/2749
👍 1
👀 1
❤️ 1
d

datajoely

08/10/2023, 8:42 AM
I’ve said we’ve needed this for a long time!
👍🏼 1
n

Nok Lam Chan

08/10/2023, 9:10 AM
Any push back in the past? I couldn't find a relevant issue.
d

datajoely

08/10/2023, 9:12 AM
No it was more that we didn’t know how to make it work
I also thin we should be able to unpack input datasets in addition to just the params
m

Matthias Roels

08/10/2023, 9:25 AM
What do you mean with “unpack datasets”?
d

datajoely

08/10/2023, 9:34 AM
Copy code
node(foo, inputs=['**dataset', 'params:bar'])
In case you provide an dictionary or iterable here
n

Nok Lam Chan

08/10/2023, 10:40 AM
isn’t dataset here a string? or a dictionary
https://docs.kedro.org/en/stable/nodes_and_pipelines/nodes.html#syntax-for-input-variables Otherwise passing a dict should work already (although we didn’t promote a lot this syntax and I found many people not aware of this)
d

datajoely

08/10/2023, 11:13 AM
I think i passed a list in memmory from one node and wanted to pass it as kwargs to another I would want to use **
m

marrrcin

08/11/2023, 6:25 AM
That would be a nice feature!
👍🏼 1
n

Nok Lam Chan

08/11/2023, 12:43 PM
Could I get your upvote in the Github issue if you think this is useful? It helps to prioritise it. 🙏🏼 Bonus if you can give an example why this is useful to you.
5 Views