parse pipeline argument string 'None' to None

When definining a kfp pipeline, with an str argument, if the value is ‘None’, then the component recieve ‘None’. Wouldn’t be better, to parse str ‘None’ to None (NoneType)?

Now:

@component
def task1(arg1: str = None):
  if arg1 is None or arg1 == 'None':
    print('arg1 is None')
@pipeline
  pipeline(arg1: str = 'None'):
  task1(arg1)

expected:

@component
def task1(arg1: str = None):
  if arg1 is None:
    print('arg1 is None')
@pipeline
  pipeline(arg1: str = 'None'):
  task1(arg1)

[chensun]

the parsing from ‘None’ to None, should be done (only) inside the py component?

No, I don’t think we should ever convert 'None' to None. It could be a legit user intention that they may want to pass 'None' and consume it as string. We don’t want to be “oversmart”.

Or not parsing at all, and that py component should recieve ‘None’ and be treated by the user inside the function? (option a)

Yes, users can choose whatever they want, 'None', 'null', 'NoValue', etc. The system doesn’t need to be aware of the user-chosen contract.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.