Managing CoolProp interface dependencies - Tree structure about externalmedia

[slamer59]

I update Build-CMake.sh

Compilation seems to work fine pointing to v6.1.0 tag of CoolProp

[casella]

I'm not much into this kind of technical details, but I would favour approach n. 3. Having CoolProp as a submodule or subtree of ExternalMedia is wrong from a logical point of view. It would be like having the United States as a province of Italy 😄

Furthermore, if in the future people want to add more solvers in a clean way, 3. seems much better than 1. and 2., otherwise we end up also adding China, India etc. as other provinces of Italy...

Ideally, it should be possible for anyone to choose which solvers should be actually compiled into the library, possibly including their private own one, and this seems to be easier to do with 3.. We will provide pre-compiled libraries with RefProp and CoolProp support, but that's by no means exhaustive of what you can do with this library.

[bilderbuchi]

I have no strong feelings in either direction (@casella's reasoning above seems sound, though). I only caution that many of the git submodule problems have stemmed from things since fixed in current-ish git versions, so I would de-emphasize the weight of old articles on the internet.

[jowr]

I may have some time next year - I am going to fix this by including External CMake project using ExternalProject_Add and an option to disable CoolProp.

Please let me know if working on this is obsolete: @slamer59 and @casella

[slamer59]

Hello,
I move to others initiatives. For me coolprop is still what is lacking inside open source modelica. It has been hard to compiled it and difficult to maintain and each time someone asked for help nobodies was there unfortunately (me included).

So... I think it worth it to make it clean and good documentation on how it works and focus on OSS.

[jowr]

Fine I'll have a look at it.

I may reach out at some point and ask for a review of my changes - I hope that is OK.

[casella]

I may have some time next year - I am going to fix this by including External CMake project using ExternalProject_Add and an option to disable CoolProp.

Please let me know if working on this is obsolete: @slamer59 and @casella

I've been playing around with the idea of using the pre-compiled CoolProp dll, instead of compiling it from source. This makes the project management a lot easier. I made some proof-of-concept experiments and it seems to work fine.

@jowr, I should have time to work on this during or immediately after the Christmas break, maybe on a branch. What do you think?

[casella]

See discussion here

[jowr]

This has been foxed for v3.3.0