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

Bicubic backend and first_saturation_deriv #713

Closed
Astrinus opened this issue Jul 7, 2015 · 3 comments
Closed

Bicubic backend and first_saturation_deriv #713

Astrinus opened this issue Jul 7, 2015 · 3 comments
Milestone

Comments

@Astrinus
Copy link

Astrinus commented Jul 7, 2015

Fluid: R245fa
Language: C++
(P,Q) = (406200, 0)

Using the HEOS backend, saturation derivatives are fine, whereas using the BICUBIC&HEOS one results in "Invalid index to calc_first_saturation_deriv in TabularBackends".

@jowr
Copy link
Member

jowr commented Jul 7, 2015

I am not totally sure, but I think everything is fine. They are simply not implmented.

@Astrinus
Copy link
Author

Astrinus commented Jul 7, 2015

Either in CoolProp 5.1.0 and in the 5.1.1dev of a month ago they are implemented.

Il 07/07/2015 13:24, Jorrit Wronski ha scritto:

I am not totally sure, but I think everything is fine. They are simply not
implmented.

— Reply to this email directly or view it on GitHub
#713 (comment).

@ibell
Copy link
Contributor

ibell commented Jul 7, 2015

Probably just a bug. I thought I had tests for the derivatives with the
tables, but let's see. Should be pretty straightforward.

On Tue, Jul 7, 2015 at 6:16 AM, Astrinus notifications@github.com wrote:

Either in CoolProp 5.1.0 and in the 5.1.1dev of a month ago they are
implemented.

Il 07/07/2015 13:24, Jorrit Wronski ha scritto:

I am not totally sure, but I think everything is fine. They are simply
not
implmented.

— Reply to this email directly or view it on GitHub
<#713 (comment)
.


Reply to this email directly or view it on GitHub
#713 (comment).

@ibell ibell closed this as completed in 11417d3 Jul 8, 2015
@ibell ibell added this to the v5.1.1 milestone Jul 8, 2015
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

3 participants