Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update interface for newer CoolProp version 6.1.X #3

Closed
slamer59 opened this issue Oct 20, 2017 · 6 comments
Closed

Update interface for newer CoolProp version 6.1.X #3

slamer59 opened this issue Oct 20, 2017 · 6 comments

Comments

@slamer59
Copy link

Hello,
the wrapper with ExternalMedia is not maintained anymore here since 3 years.

The wrapper seemed to be in sync with version 4.2.x of coolprop.

There is precompiled librarie for windows but I was not able to compile it directly from source. Some simple update have to be made in this libraries. Maybe it is a good idea to update it ? Integrate it with CoolProp ?

I will open the same issue in CoolProp.

The work will be :
-> make to CMake (easier to make cross compilation)
-> change functions from v4.X to 6.X

What do you think ?
I may take some times to make this change with some helps.

Thomas

@bilderbuchi
Copy link
Collaborator

See CoolProp/ExternalMedia#13 (comment) and following for an update of the situation.

@jowr
Copy link
Collaborator

jowr commented Oct 23, 2017

You have seen this repository, right? https://github.com/CoolProp/ExternalMedia

@slamer59
Copy link
Author

After bilderbuchi comment I saw that. Thanks for the update. I will follow comments on this topic.

@thorade
Copy link
Contributor

thorade commented Oct 26, 2017

Also see the summary in the wiki:
https://github.com/modelica/ExternalMedia/wiki

@casella
Copy link
Collaborator

casella commented Nov 20, 2017

OK, I apologize for the lazy management, trying to figure out how to deal with this project properly was not trivial, due to multiple conflicting requirements.

I had some ideas how to proceed that I wrote in the wiki in October, but after prototyping work and some discussion with @adrpo I have changed my mind a bit.

I tried to summarize the situation and requirement and to come up with a detailed action plan in issue #4, please comment there

@casella
Copy link
Collaborator

casella commented Nov 22, 2017

The discussion continues in #4

@casella casella closed this as completed Nov 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants