search

ibell / coolprop

Deprecated version - go to
https://github.com/CoolProp/CoolProp
MIT License
24 stars 16 forks source link

Excel Addin: R404A wrong thermal conductivity #232

Closed spiderpinne closed 10 years ago

spiderpinne commented 10 years ago

The Coolprop 4.2.0 Excel Addin returns thermal conductivity for R404A in the wrong decimal power.

The correct Value would be the returned value * 10 ^ 3.

ibell commented 10 years ago

I made the fix locally, but didn't push it. I'll release 4.2.4 tonight with this bug fixed. Alternatively you can build the DLL yourself.

On Mon, Jul 28, 2014 at 10:43 AM, spiderpinne notifications@github.com wrote:

The Coolprop 4.2.0 Excel Addin returns thermal conductivity for R404A in the wrong decimal power.

The correct Value would be the returned value * 10 ^ 3.

— Reply to this email directly or view it on GitHub https://github.com/ibell/coolprop/issues/232.

ibell commented 10 years ago

Build in progress, should be online in another hour or two.

On Mon, Jul 28, 2014 at 10:56 AM, Ian Bell ian.h.bell@gmail.com wrote:

I made the fix locally, but didn't push it. I'll release 4.2.4 tonight with this bug fixed. Alternatively you can build the DLL yourself.

On Mon, Jul 28, 2014 at 10:43 AM, spiderpinne notifications@github.com wrote:

The Coolprop 4.2.0 Excel Addin returns thermal conductivity for R404A in the wrong decimal power.

The correct Value would be the returned value * 10 ^ 3.

— Reply to this email directly or view it on GitHub https://github.com/ibell/coolprop/issues/232.

ibell commented 10 years ago

Ok its online now. Can you please confirm that this fixes it?

On Mon, Jul 28, 2014 at 11:01 AM, Ian Bell ian.h.bell@gmail.com wrote:

Build in progress, should be online in another hour or two.

On Mon, Jul 28, 2014 at 10:56 AM, Ian Bell ian.h.bell@gmail.com wrote:

I made the fix locally, but didn't push it. I'll release 4.2.4 tonight with this bug fixed. Alternatively you can build the DLL yourself.

On Mon, Jul 28, 2014 at 10:43 AM, spiderpinne notifications@github.com wrote:

The Coolprop 4.2.0 Excel Addin returns thermal conductivity for R404A in the wrong decimal power.

The correct Value would be the returned value * 10 ^ 3.

— Reply to this email directly or view it on GitHub https://github.com/ibell/coolprop/issues/232.

spiderpinne commented 10 years ago

Fix works! Thank you very much for the fast response!