Hey guys, I'm running into an issue with registering a flow that uses CronClock with parameter default inputs. On v. 0.15.5

ValueError: Flow could not be deserialized successfully. Error was: ValidationError({'schedule': {'clocks': {0: {'parameter_defaults': defaultdict(<class 'dict'>, {'accnt': {'value': ['Field may not be null.']}, ....

std_inputs = { 'report':False,
               'accnt':               None,  # string - delimited list
               'strategy':            None,  # string - delimited list
               'institutionalAdvisor':None,  # string - delimited list
               'endDate':             None,  # string - 'YYYY-MM-DD'
               'shadow':              None,  # SPY, treasury
               'benchmark':           None,  # SPY, PUT, BXM
               'fees':                False,  # Bool
               'output':              'daily'  # monthly, daily
               }

with Flow('Update Performance tables',
        schedule = Schedule(
                clocks = [
                      CronClock("7 6 * * 1-5",
                                start_date=pendulum.now("America/Chicago"),
                                parameter_defaults = std_inputs)]...

Any help would be greatly appreciated!

How did you define the

Parameter

? Is it required and what is the default? Looking for

accnt

specifically

Hi @chicago-joe - it looks like you have a required parameter on your flow

accnt

that you’re setting to

None

in your parameter defaults. For any required parameters you’ll need to set a default there

@Kevin Kho parameter defined as:

accnt = Parameter('accnt', default = None)

@nicholas is correct, but this is intentional, as this script is a performance reporting module that takes a variety of inputs, where you can view the report for an account, for multiple accounts, or even entire strategies as whole. This input = None by default is by design on our side

Wouldnt that effectively be

required=False

if you can bypass it?

tbh, I wasn't quite clear on what happens when you set required=False. Prefect Github:

Args:

- name (str): the Parameter name.

- default (any, optional): A default value for the parameter. Must be a JSON-Serializable type.

- required (bool, optional): If True, the Parameter is required and the

default value is ignored. Defaults to

False

if a

default

is

provided, otherwise

True

.

You can go for it. Doc changes are certainly welcome

sweeet I'll be on that later this week!