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

add butterfly valve model #975

Closed
Mathadon opened this issue Jun 30, 2018 · 6 comments · Fixed by #1441
Closed

add butterfly valve model #975

Mathadon opened this issue Jun 30, 2018 · 6 comments · Fixed by #1441
Assignees

Comments

@Mathadon
Copy link
Member

Mathadon commented Jun 30, 2018

We currently do not have a model for a butterfly valve. Typically these valves are used for open/closed operation only, however in some cases they are used for control too. In these cases it would be good to have a somewhat accurate valve characteristic.

Some sources for such characteristics:

Some questions before I implement this:

  • I propose to implement the characteristic by extending TwoWayPolynomial.
  • We could add the valve diameter as parameter that computes a default value for dp_nominal, using another polynomial e.g. based on the data in https://www.engineeringtoolbox.com/butterfly-valves-flow-coefficients-d_224.html , or http://cgproducts.johnsoncontrols.com/MET_PDF/347VB2.PDF?x=86 page 22.
  • The cleanest way to do this would be to add the option Diameter to CvData, but this would be an invalid option for the other valve models so I don't think is realistic. Unless an enumeration can be extended and redeclared. So perhaps it's better to not add the diameter option for now, or to implement it as a separate function.
  • Proposed name: TwoWayButterfly

@icupeiro @damienpicard this is for Infrax

@Mathadon Mathadon self-assigned this Jun 30, 2018
@mwetter
Copy link
Contributor

mwetter commented Jun 30, 2018

@Mathadon Using TwoWayPolynomial as the base class makes sense.
How would one usually parameterize the valve? Is it common to simply use the diameter, or does one really specify the Cv value for butterfly valves? If it is the first, maybe it needs a different base class that does not use ValveParameters.

@Mathadon
Copy link
Member Author

Mathadon commented Jul 1, 2018

I think butterfly valves are typically selected such that they match the pipe diameter, since that makes them easy to install.

It looks quite tedious to do the new implementation without using ValveParameters and without duplicating code. Perhaps it's better to use ValveParameters and to assign final default values for Kv, Cv, etc?

Mathadon added a commit that referenced this issue Jul 7, 2018
@Mathadon
Copy link
Member Author

Mathadon commented Jul 7, 2018

In 27e0682 I implemented figure 10 from http://www.valmatic.com/pdfs/SurgeControlinPumpingSystems3-17-09.pdf. I should still verify the shape against other sources.

@Mathadon
Copy link
Member Author

Mathadon commented Jul 7, 2018

Butterfly valve in brown:

schermafbeelding 2018-07-07 om 19 49 45

@Mathadon
Copy link
Member Author

Could compute Kv from DN value, but sources disagree quite a bit on the pressure drop at fully opened position. That makes sense since some butterfly valves may be thinner than others. However, the Kv curve should be consistent with the Kv value since at closed position all valves probably behave more similarly.

http://www.valvias.com/flow-coefficient-butterfly-valve.php
http://www.mydatabook.org/fluid-mechanics/flow-coefficient-opening-and-closure-curves-of-butterfly-valves/

Mathadon added a commit that referenced this issue Dec 20, 2020
@Mathadon
Copy link
Member Author

Commit ceadd7b revises the implementation to use a different curve and matching DN->Kv curve based on the data in http://www.mydatabook.org/fluid-mechanics/flow-coefficient-opening-and-closure-curves-of-butterfly-valves/. The curve difference is quite large (see image below) but it's hard to find a good source and clearly these valves have different shapes in practice too, which cause different flow profiles.

image

Mathadon added a commit that referenced this issue Dec 22, 2020
added figure in the docs
for #975
Mathadon added a commit that referenced this issue Dec 22, 2020
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

Successfully merging a pull request may close this issue.

2 participants