What is the purpose of `AbstractTOMLDict`?

CliMA / ClimaParams.jl

Contains all universal constant and physical parameters in CLIMA

Apache License 2.0

36 stars 6 forks source link

Open Sbozzolo opened 5 months ago

Sbozzolo commented 5 months ago

The documentation reads:

"""
    AbstractTOMLDict{FT <: AbstractFloat}

Abstract parameter dict. One subtype:
 - [`ParamDict`](@ref)
"""

Indeed, there is only one subtype and the rest of the package is implemented around this type.

Maybe we can we remove this extra layer in the next breaking release?

nefrathenrici commented 5 months ago

There used to be an additional subtype, but now this abstract type is unneeded. I can remove it soon.