If Kedro supports unpacking parameters dictionary,...
# user-researchn
Nok Lam Chan
08/09/2023, 8:26 PMIf 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 AMI’ve said we’ve needed this for a long time!
👍🏼 1
n
Nok Lam Chan
08/10/2023, 9:10 AMAny push back in the past? I couldn't find a relevant issue.
d
datajoely
08/10/2023, 9:12 AMNo it was more that we didn’t know how to make it work
datajoely
08/10/2023, 9:13 AMI also thin we should be able to unpack input datasets in addition to just the params
m
Matthias Roels
08/10/2023, 9:25 AMWhat do you mean with “unpack datasets”?
d
datajoely
08/10/2023, 9:34 AM Copy code
node(foo, inputs=['**dataset', 'params:bar'])n
Nok Lam Chan
08/10/2023, 10:40 AMisn’t dataset here a string? or a dictionary
Nok Lam Chan
08/10/2023, 10:40 AMhttps://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 AMI 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 AMThat would be a nice feature!
👍🏼 1
n
Nok Lam Chan
08/11/2023, 12:43 PMCould 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.
6 Views