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

ThreeWayTable Valve #1227

Closed
alexanderAKU opened this issue Oct 29, 2019 · 5 comments · Fixed by #1245
Closed

ThreeWayTable Valve #1227

alexanderAKU opened this issue Oct 29, 2019 · 5 comments · Fixed by #1245

Comments

@alexanderAKU
Copy link
Contributor

The library contains a two way valve that has a table based flow characteristic (TwoWayTable). However, a three way valve with table-based characteristics does not exist. I tried to model a three way table-based valve in the same way as the model ThreeWayLinear or ThreeWayEqualPercentageLinear: I tried to redeclare the partial two way valve (res1 and res3) and use the TwoWayTable. However, this approach does not work because the parameter l in the partial valve (res1 and res3) is final as well as the parameter l in the model TwoWayTable.

I think it would be a good approach to release the "final" modifier in the PartialThreeWayValve because this model is a base class. The final modifier could be added in the child classes if needed.
Are there any suggestions how to solve this issue?

Best regards

@Mathadon
Copy link
Member

Mathadon commented Nov 1, 2019

It's not clear to me why you have to modify the parameter l, can you elaborate on that?

@alexanderAKU
Copy link
Contributor Author

The replaceable valves res1 and res3 in the model PartialThreeWayValve include the final parameter l. The parameter l of in the model TwoWayTable is final too. Thus, res1 and res3 can not be replaced by the model TwoWayTable because the parameter l would have two different final values.

mwetter added a commit that referenced this issue Nov 5, 2019
@mwetter
Copy link
Contributor

mwetter commented Nov 5, 2019

@alexanderAKU : Would the commit 0e647cf address what you need?
@Mathadon : This would be backward compatible, and not impact the dialog window of Dymola.

@Mathadon
Copy link
Member

Mathadon commented Nov 6, 2019

Ok I get it, the problem is that the PartialThreeWayValve wants to assign a value of l that conflicts with the definition of TwoWayTable.

The proposed change looks good!

@alexanderAKU
Copy link
Contributor Author

@mwetter the commit solves my problem. Thanks.

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.

3 participants