Surprising default combining rule in association term about clapeyron.jl HOT 3 OPEN

On the documentation, admittedly, it's not in the most obvious place, but we did document the fact that we do have no combining rule is the default: https://clapeyronthermo.github.io/Clapeyron.jl/dev/api/parameters/#Clapeyron.AssocOptions.

As for what the default should be, the difficulty is that did SAFT equations use different combining rules (I believe that SAFT-VR Mie uses Elliott whereas CPA using CR1. When it comes to specifying association parameters though, I do believe that it is better to leave the combining rule off by default as, when you get to SAFT-VR Mie / gamma Mie where you have H, e1, e2 and a1 association sites where you don't necessarily expect cross-association between everything, it's safer to leave it up to the user to specify.

I will add some docs on that in the documentation that I'm updating.

from clapeyron.jl.

Happy to report the updated docs do explicitly highlight how the association interactions can be adjusted: https://github.com/ClapeyronThermo/Clapeyron.jl/blob/docs-update/docs/src/tutorials/basics_model_construction.md

from clapeyron.jl.

If you explicitly select CPA/sCPA, I feel like the default sent to the association term should be CR1. Or at least, it should be clearly documented in the model documentation for all the relevant models that the combining rule is not set at all. I see your point about the nuances of how the association term gets used in other places, and sympathize. I'm struggling with the same thing in teqp now.

from clapeyron.jl.