Join the Kedro community

Node definition requires name parameter

hey all,

I noticed something that when i define the following node:

node(
    func=sample_func,
    # name="sample_func"    -> does not work when this is commented
    inputs="epubdf",
    outputs="result"
)

and then run this test:

kedro_session.run(node_names=['sample_func'])

I get an error saying name doesnt exists but when i specify name parameter in node itself it works. I thought if i don't provide it, name would be equal to func name itself, no?

9 comments

MMerel Theisen

Node name is optional, so if you haven't specified it, it will be None

FFazil Topal

I see, would it make sense to make it default to func.__<i><code>name</code></i>__ ? Or was there a reason why it wasn't?

MMerel Theisen

The name was just added for visualisation in Kedro Viz and has no functional purpose.

FFazil Topal

hmm but session run uses it somehow. if i wanna run a single node in my pipeline, it breaks

JJuan Luis Cano Rodríguez

would it make sense to make it default to func.__<i><code>name</code></i>__ ?

it's been discussed, it's a backwards-incompatible change but I actually think we should do it, have a look at https://github.com/kedro-org/kedro/issues/3575

FFazil Topal

Thanks, that's what i hit now with not able to run a single node unless name param is specified. I also run the kedro registry describe __ default __ which returns func names so i thought it was working but tests fail. Reading the thread, i also see it has other issues with default as well so i guess for now i'll put the names manually 😄

NNok Lam Chan

The simple reason for this is, you can use the same function for different node and that default wouldn't work.

NNok Lam Chan

Kedro need unique nodes to sort out the execution orders, so behind the scene it generate some default name to make this work.

You will get error if you try to define the same node name for more than 1 node too.

FFazil Topal

Makes sense. Thanks for the explanation

Add a reply

Join on Slack