Dseal95 / AGA8-Detail

Python Implementation of AGA8 DETAIL method for calculating gas compressibility factor, Z from P, T and gas composition.
4 stars 3 forks source link

PropertiesDetail() implemented or not? #2

Open philip-lindner opened 5 months ago

philip-lindner commented 5 months ago

Hi @Dseal95 ,

first of all: Thanks for sharing your Python Implementation! This could be of great use for my project!

There is just one thing I do not understand: In the README it says, that PropertiesDetail() is not implemented because you were only interested in Z. However, I do see the function properties_detail() defined in AGA8Detail.py . In addition, I see that the the output of cv,cp, and other quantities are tested in test_AGA8Detail.py.

Could you clarify whether properties_detail() is implemented and the respective parameters can be used?

Best regards, Philip

Dseal95 commented 5 months ago

Hi Philip,

That’s a good spot. Initially I hadn’t implemented it but later did and I forgot to update the README.md.

It is implemented and will be run automatically after executing the run() method.

Regards, Dan

On Fri, Apr 19, 2024 at 09:05 philip-lindner @.***> wrote:

Hi @Dseal95 https://github.com/Dseal95 ,

first of all: Thanks for sharing your Python Implementation! This could be of great use for my project!

There is just one thing I do not understand: In the README it says, that PropertiesDetail() is not implemented because you were only interested in Z. However, I do see the function properties_detail() defined in AGA8Detail.py . In addition, I see that the the output of cv,cp, and other quantities are tested in test_AGA8Detail.py.

Could you clarify whether properties_detail() is implemented and the respective parameters can be used?

Best regards, Philip

— Reply to this email directly, view it on GitHub https://github.com/Dseal95/AGA8-Detail/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/APIIZOBOD42CFE32BY2PZZTY6DF4HAVCNFSM6AAAAABGOWL4MWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TEMZVGEYDMMY . You are receiving this because you were mentioned.Message ID: @.***>