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

SRK not reading critical temperature (Tc) #175

Closed
NicolasHeredia opened this issue Apr 20, 2023 · 7 comments
Closed

SRK not reading critical temperature (Tc) #175

NicolasHeredia opened this issue Apr 20, 2023 · 7 comments

Comments

@NicolasHeredia
Copy link

Hello and thank you in advance.

I'm trying to create an SRK model with a custom database.

model_nh = SRK(["metanol", "benceno"]; userlocations=["database/"], alpha_userlocations = ["database/"])

And I did get:

cannot found any values for Tc.

Searching the SRK source code I found that the "userlocations" argument is not being passed to RK.
https://github.com/ClapeyronThermo/Clapeyron.jl/blob/4136658d31edbbfde794fbefd588024d9ddd79a2/src/models/cubic/RK/variants/SRK.jl

Could that be the problem?

@longemen3000
Copy link
Member

yes, you are right!
in the meantime, you could use RK(["metanol", "benceno"]; userlocations=["database/"], alpha = SoaveAlpha, alpha_userlocations = ["database/"])

longemen3000 added a commit that referenced this issue Apr 20, 2023
@pw0908
Copy link
Member

pw0908 commented Apr 20, 2023

Is this problem unique to SRK?

@longemen3000
Copy link
Member

i remember doing a check a while ago for derived models, it seems that SRK just passed through the cracks

@pw0908
Copy link
Member

pw0908 commented Apr 20, 2023

QCPR might have that problem?

@longemen3000
Copy link
Member

not directly, the userlocations got added here:

QCPR_userlocations = vcat("@REMOVEDEFAULTS","@DB/cubic/QCPR/QCPR_critical.csv", "@DB/cubic/QCPR/QCPR_unlike.csv",userlocations)
, but verbose was not considered

@pw0908
Copy link
Member

pw0908 commented Apr 20, 2023

Cool, so we can close this issue now

@NicolasHeredia
Copy link
Author

Thank you for your answers

@pw0908 pw0908 closed this as completed Apr 21, 2023
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