Hyperparameters should be a mutable struct (not a named tuple)

[DilumAluthge]

That way we can do hyperparameter optimization with MLJ.

[cscherrer]

Let's make it a Dict{Symbol, Any} or similar, then we can easily convert to a named tuple to use in the Soss model

[cscherrer]

Oh but does it have to be a mutable struct for MLJ? Maybe we just make a little wrapper? Would be annoying for users to have to do it from scratch each time

[DilumAluthge]

Yeah MLJ needs to be able to access with getproperty and setproperty!

What if we make a mutable struct that wraps a Dict{Symbol, Any}, and converts getproperty and setproperty! to the appropriate Dict methods.

[cscherrer]

Yep, I like it.

[DilumAluthge]

https://github.com/JuliaCollections/PropertyDicts.jl

[DilumAluthge]

Currently https://github.com/JuliaCollections/PropertyDicts.jl already supports getproperty.

Once that package is registered, we could make a PR to add support for setproperty! for Symbol keys.