Higor Carmanini
07/17/2023, 7:26 PMpylance
incorrectly inferring that the pipeline
function (as imported from kedro.pipeline
is actually a module. It gets in the way of showing the proper documentation for kedro.pipeline.modular_pipeline.pipeline()
, and I figure could turn some less Kedro-savvy devs away by thinking they're doing it wrong (me a while back 🙃)Nok Lam Chan
07/17/2023, 8:05 PMNok Lam Chan
07/17/2023, 8:06 PMHigor Carmanini
07/17/2023, 8:12 PMpipeline
was not a module being called.Nok Lam Chan
07/17/2023, 8:16 PMHigor Carmanini
07/17/2023, 8:29 PMpipeline.py
module inside kedro.pipeline
, so the clash is not that much of a surprise now.
Could be the static checker's lookup priority/order, if that makes sense. There's a reference ambiguity when it comes to kedro.pipeline.pipeline
.Higor Carmanini
07/17/2023, 8:31 PMpipeline
function name to a more descriptive build_pipeline
or anything like that. As a kicker we'd get a more description function name, but I guess this is not on the table 🙂Nok Lam Chan
07/18/2023, 12:13 PMHigor Carmanini
07/18/2023, 8:11 PMHigor Carmanini
07/18/2023, 8:11 PMpipeline
which it thinks is the kedro.pipeline.pipeline
moduleNok Lam Chan
07/18/2023, 11:34 PMNok Lam Chan
07/18/2023, 11:44 PMHigor Carmanini
08/02/2023, 10:47 PMpylance
:
"python.analysis.typeCheckingMode": "basic",
"python.analysis.autoFormatStrings": true,
"python.analysis.enablePytestSupport": true,
// "python.analysis.inlayHints.callArgumentNames": true,
"python.analysis.inlayHints.functionReturnTypes": true,
"python.analysis.inlayHints.pytestParameters": true,
"python.analysis.inlayHints.variableTypes": false,
"python.analysis.useLibraryCodeForTypes": true,
Nok Lam Chan
08/02/2023, 11:04 PM