Should create_*_parameters simply be constructors?

[glwagner]

There are many functions that take forms like

SurfaceFluxes.jl/parameters/create_parameters.jl

Line 44 in 10c2840

function create_uf_parameters(toml_dict, ::UF.ChengType)

It seems these are more straightforwardly defined as constructors, eg:

function ChengParams(toml_dict::Dict)

(If there is a better way to identify toml_dict than just ::Dict, even better.)

A second slightly related question is: why are these not in the source code? This leads to anti-patterns like

include(joinpath(pkgdir(SurfaceFluxes), "parameters", "create_parameters.jl"))

which I think should be discouraged?

[glwagner]

cc @kmdeck

[glwagner]

Note this would presumably obviate the need for the extraneous UF.ChengType, for example, unless I'm missing something.