Skip to content
This repository has been archived by the owner on Mar 29, 2019. It is now read-only.

Update readme.md #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update readme.md #26

wants to merge 1 commit into from

Conversation

tonkiplis
Copy link

Added comments for an integration of refprop9.1 with matlab 64 bits

my system:

MAC OS X 10.8.5
X-Code 5.1.1 with its command-line developer tool
gcc: Apple LLVM version 5.1 (clang-503.0.40) (based on LLVM 3.4 svn)
gfortran : 4.8.1
MATLAB_R2011a 64 bit installation

Hope this can help,

Tonkiplis

@jowr
Copy link
Owner

jowr commented Apr 24, 2015

Thanks for the quick action. Are you familiar with the markdown language used in the readme? You can preview your changes at https://github.com/tonkiplis/librefprop.so/tree/patch-1. It would be very nice if you could integrate them a bit better with the rest of the code and the docs.

@tonkiplis
Copy link
Author

I am not sure what you mean with integrate them a bit better with the rest of the code. I do not think it was a 100% success as I had to symlink the wrong version of the gcc comiler. So it is still 100% experimental.

If we go for a better integration, it means I have to work on the former comments by nkampyhttps://github.com/nkampy and speredennhttps://github.com/speredenn, is this allowed?

Have a nice day,

Thierry

From: Jorrit Wronski [mailto:[email protected]]
Sent: Freitag, 24. April 2015 20:02
To: jowr/librefprop.so
Cc: Meier Thierry
Subject: Re: [librefprop.so] Update readme.md (#26)

Thanks for the quick action. Are you familiar with the markdown language used in the readme? You can preview your changes at https://github.com/tonkiplis/librefprop.so/tree/patch-1. It would be very nice if you could integrate them a bit better with the rest of the code and the docs.


Reply to this email directly or view it on GitHubhttps://github.com//pull/26#issuecomment-96014659.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants