Generalize friction corrected Isentr nozzle flow model

[davideziviani]

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

[ibell]

Can you explain what you have in mind?

…

On Nov 29, 2016 6:54 AM, "dziviani" ***@***.***> 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 <#24>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABxha1OJfBgKMv3X1qpV8UKSzlwRqaCuks5rDBJ_gaJpZM4K-4rm> .

[davideziviani]

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?

[ibell]

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" ***@***.***> 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 <#24 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABxha00q0phj1J9xM65bpsabqCYiTxfDks5rDCI5gaJpZM4K-4rm> .

[davideziviani]

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.

[ibell]

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" ***@***.***> 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 <#24 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABxha5VK30izHm6ErNiJEhkgf-7fANy0ks5rDCPUgaJpZM4K-4rm> .

[davideziviani]

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