Alex Bain
04/27/2022, 5:28 PMv0.19.3
to new v1.0.0
and Flyte Admin no longer wants to register workflows like:
@flytekit.workflow
def DDPTrainingWorkflow(
train_datasets: typing.List[str] = [],
...
):
patch_experiment(train_datasets)
@flytekit.task()
def patch_experiment(
train_datasets: typing.List[str],
)
^^^^
while registering the workflow, this throws:
[2022-04-27T15:09:25Z] details = "Type mismatch for Parameter train_datasets in default_inputs has type collection_type:<simple:STRING > , expected simple:NONE "
clearly something doesn't like the default []
value of train_datasets
against its declared type typing.List[str]
. Ideas?katrina
Alex Bain
04/27/2022, 5:40 PMflyte-cli register-files
to register this workflow (with flytekit 0.23.0b6). Update - I am able to reproduce this in another repo where I have flytekit 0.32.3 and I use the latest flytectl.Haytham Abuelfutuh
Alex Bain
04/27/2022, 6:18 PM0.23.0b6
and 0.32.3
. I can't easily upgrade to flytekit 1.0.0 yet.Ketan (kumare3)
Alex Bain
04/27/2022, 6:19 PM@flytekit.workflow
def DDPTrainingWorkflow(
train_datasets: typing.List[str] = ???, # was previously []
):
Ketan (kumare3)
x: typing.List[str] = []
should workEduardo Apolinario (eapolinario)
04/27/2022, 6:36 PMKetan (kumare3)
Alex Bain
04/27/2022, 10:05 PMEduardo Apolinario (eapolinario)
05/06/2022, 12:58 AMAlex Bain
05/09/2022, 7:49 PMEduardo Apolinario (eapolinario)
05/09/2022, 10:24 PMAlex Bain
05/09/2022, 10:43 PMNone
default values for collection types (e.g. []
and {}
). Thanks for following up.Eduardo Apolinario (eapolinario)
05/10/2022, 8:20 PM