Iñigo Hidalgo
05/24/2024, 2:08 PMdatajoely
05/24/2024, 2:08 PMIñigo Hidalgo
05/24/2024, 2:08 PMdatajoely
05/24/2024, 2:31 PMIñigo Hidalgo
05/24/2024, 2:32 PMdatajoely
05/24/2024, 2:35 PMgraphlib
which was added to the python std lib after Kedro was initially released
https://github.com/kedro-org/kedro/pull/3728/filesIñigo Hidalgo
05/24/2024, 2:38 PMNok Lam Chan
05/24/2024, 2:41 PMNok Lam Chan
05/24/2024, 2:43 PMdatajoely
05/24/2024, 2:44 PMIñigo Hidalgo
05/24/2024, 2:44 PMIñigo Hidalgo
05/24/2024, 2:48 PMno guarantee adding a silo node will execute first or notwould you expect a node with no input dependencies to run before every node with dependencies, or not that either? i feel like i just read that in either your or Joel's links
Iñigo Hidalgo
05/24/2024, 2:50 PMNok Lam Chan
05/24/2024, 2:51 PMwould you expect a node with no input dependencies to run before every node with dependencies, or not that either? i feel like i just read that in either your or Joel's linksprobably yes, at least in the old
toposort
(in 0.19.3 it switches to graphlib so I am less familiar with). The first step of toposort is sort nodes into nodes group, and nodes without input dependencies will be executed first.Nok Lam Chan
05/24/2024, 2:54 PMI guess this expectation was removed in this later PR?I am not sure, is there something characteristics that you need to rely on? I will probably check
group_nodes
insteadIñigo Hidalgo
05/24/2024, 2:58 PMNok Lam Chan
05/24/2024, 3:02 PM