pdsim icon indicating copy to clipboard operation
pdsim copied to clipboard

Generalize friction corrected Isentr nozzle flow model

Open davideziviani opened this issue 9 years ago • 6 comments

Allow to specify the calibrated coefficients if needed to apply the flow model to different compressors/expanders

davideziviani avatar Nov 29 '16 11:11 davideziviani

Can you explain what you have in mind?

On Nov 29, 2016 6:54 AM, "dziviani" [email protected] wrote:

Allow to specify the calibrated coefficients if needed to apply the flow model to different compressors/expanders

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ibell/pdsim/issues/24, or mute the thread https://github.com/notifications/unsubscribe-auth/ABxha1OJfBgKMv3X1qpV8UKSzlwRqaCuks5rDBJ_gaJpZM4K-4rm .

ibell avatar Nov 29 '16 12:11 ibell

Yes sure. I was working on the Z-compressor model and I am running the detailed flow model for two flow paths to determine the friction flow correction factor. As I am using the same fitting function you adopted in the hybrid leakage model, it is useful to enable the user to specify the a_0,...,a_10, Re_star coefficients. Similar story applies for example to rolling piston compressor etc.

Does this make sense?

davideziviani avatar Nov 29 '16 13:11 davideziviani

I guess so, but that seems like a lot of adjustable parameters ! Would you not be better off adjusting the flow path length or a general flow coefficient?

On Nov 29, 2016 8:01 AM, "dziviani" [email protected] wrote:

Yes sure. I was working on the Z-compressor model and I am running the detailed flow model for two flow paths to determine the friction flow correction factor. As I am using the same fitting function you adopted in the hybrid leakage model, it is useful to enable the user to specify the a_0,...,a_10, Re_star coefficients. Similar story applies for example to rolling piston compressor etc.

Does this make sense?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ibell/pdsim/issues/24#issuecomment-263564221, or mute the thread https://github.com/notifications/unsubscribe-auth/ABxha00q0phj1J9xM65bpsabqCYiTxfDks5rDCI5gaJpZM4K-4rm .

ibell avatar Nov 29 '16 13:11 ibell

yup that is also true but in the case a user wanna use the friction corrected isentropic nozzle for a particular path of interest, currently it is not readily possible (let's say parametric studies on different path length and gap size). One has to change the coefficients first, and the recompile PDSim.

davideziviani avatar Nov 29 '16 13:11 davideziviani

That is true. I wonder about the speed if you are always passing the coefficients. Probably better to come up with a callable class so the coefficients can be cached.

On Nov 29, 2016 8:08 AM, "dziviani" [email protected] wrote:

yup that is also true but in the case a user wanna use the friction corrected isentropic nozzle for a particular path of interest, currently it is not readily possible (let's say parametric studies on different path length and gap size). One has to change the coefficients first, and the recompile PDSim.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ibell/pdsim/issues/24#issuecomment-263565591, or mute the thread https://github.com/notifications/unsubscribe-auth/ABxha5VK30izHm6ErNiJEhkgf-7fANy0ks5rDCPUgaJpZM4K-4rm .

ibell avatar Nov 29 '16 13:11 ibell

yes indeed. I am gonna take a look to create a callable class to cache the coefficients

davideziviani avatar Nov 29 '16 13:11 davideziviani