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

Acentric factor of air #772

Closed
jonas-z opened this issue Aug 10, 2015 · 8 comments
Closed

Acentric factor of air #772

jonas-z opened this issue Aug 10, 2015 · 8 comments
Milestone

Comments

@jonas-z
Copy link

jonas-z commented Aug 10, 2015

Hello,

how is the acentric factor of air calculated? The output of PropsSI('acentric', 'air') is 0.0816749632704
I wonder because the acentric factors of oxygen and nitrogen are 0.0222 and 0.037200000000000004.

@ibell
Copy link
Contributor

ibell commented Aug 10, 2015

Hmm that might be a bug. It is based on the saturation pressure, and I
agree that this value doesn't seem correct to me.
On Aug 10, 2015 8:20 AM, "phileasfoggy" notifications@github.com wrote:

Hello,

how is the acentric factor of air calculated? The output of
PropsSI('acentric', 'air') is 0.0816749632704
I wonder because the acentric factors of oxygen and nitrogen are 0.0222
and 0.037200000000000004.


Reply to this email directly or view it on GitHub
#772.

@jonas-z
Copy link
Author

jonas-z commented Aug 10, 2015

OK, thanks so far.
What about the Phase? CoolProp.CoolProp.PhaseSI('P',101325,'T',300,'air') returns
'supercritical_gas' shouldn't be 'gas' the correct Phase?

@ibell
Copy link
Contributor

ibell commented Aug 10, 2015

According to our definition of phase, gas is only for temperatures below
the critical temperature. See for reference
http://www.coolprop.org/coolprop/HighLevelAPI.html#phasesi-function
On Aug 10, 2015 8:45 AM, "phileasfoggy" notifications@github.com wrote:

OK, thanks so far.
What about the Phase? CoolProp.CoolProp.PhaseSI('P',101325,'T',300,'air')
returns
'supercritical_gas' shouldn't be 'gas' the correct Phase?


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

@ibell ibell added this to the v5.1.2 milestone Aug 11, 2015
@ibell
Copy link
Contributor

ibell commented Aug 12, 2015

It's a bug, REFPROP says it should be 0.0335.

@ibell
Copy link
Contributor

ibell commented Aug 15, 2015

Actually, I am not so sure about this after all. Air is a mixture, and the acentric factor is only defined for pure fluids.

@ibell
Copy link
Contributor

ibell commented Aug 15, 2015

Perhaps we should remove it entirely.

@jonas-z
Copy link
Author

jonas-z commented Aug 15, 2015

Maybe. I used CoolProp to get the Density of air. I compared the values to the results of cubic real-gas-equations. To use the cubic real-gas-equations I had to define critical properties and an acentric factor.
I calculated the values by using the mole fraction Xi.

@ibell
Copy link
Contributor

ibell commented Aug 29, 2015

Ok, the "fix" for this problem is going to be to remove the acentric factor and make it equal to infinity. I couldn't come to an agreement about what the acentric factor for air should be, so the safest answer is: not defined.

@ibell ibell closed this as completed in 70c2209 Aug 29, 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

2 participants