fix: error on inconsistent diffusion option

[haakon-e]

This pull request introduces improved error handling and model differentiation in the update_jacobian! function within the manual_sparse_jacobian.jl file. The changes ensure that unsupported or unknown vertical diffusion types are caught early and provide clearer handling for different turbulence convection models.

Key changes:

Model differentiation and error handling:

• Added an explicit check for AbstractEDMF turbulence convection models in the conditional logic, making the code more robust and maintainable.
• Introduced an error message for unknown vertical_diffusion types, preventing silent failures and aiding debugging.

---

Anna and I ran into this else block while working with Smagorinsky diffusion for her simulation setup (i.e. yaml has vert_diff: false, and no EDMF).

It is likely due to incorrectly configured yaml file: i.e. implicit_diffusion: true is only compatible with DecayWithHeightDiffusion, VerticalDiffusion, or (I presume) EDMF-type closures. For convenience, this PR proposes to explicitly check this both during model construction, and make it clearer within the jacobian that the last block pertains to EDMF.

Closes #3360

[szy21]

Thanks, I wanted to fix this for a while. Note that you probably need implicit diffusion for running simulation with smagorinsky diffusion, so you may need to add that.