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

Modelica.Electrical.QuasiStationary.MultiPhase.Blocks.SingleToMultiPhase does not conjugate inputs #2251

Closed
JoeRiel opened this issue May 9, 2017 · 1 comment
Assignees
Labels
bug Critical/severe issue L: Electrical.QuasiStationary Issue addresses Modelica.Electrical.QuasiStationary
Milestone

Comments

@JoeRiel
Copy link
Contributor

JoeRiel commented May 9, 2017

The block is extended from ComplexSIMO:
extends Modelica.ComplexBlocks.Interfaces.ComplexSIMO(final nout=m);
As such it inherits the useConjugateInput boolean parameter. The equations for the block, however, use u rather than uInternal, so the parameter has no effect. The solution is either to extend the block to use uInternal, or pass final useConjugateInput = false in the extends statement.

The same issue exists with SymmetricalComponents, though there the parameter is a boolean vector.

@beutlich beutlich added this to the MSL_next-MINOR-version milestone May 10, 2017
@beutlich beutlich added bug Critical/severe issue L: Electrical.QuasiStationary Issue addresses Modelica.Electrical.QuasiStationary labels May 10, 2017
@christiankral
Copy link
Contributor

The parameter useConjugateInput was introduced for backwards compatibility reasons but the parameters were not set final useConjugateInput = false in the following models, as the conjugate complex input does not make sense:

  • Modelica.Electrical.QuasiStationary.MultiPhase.Blocks.SymmetricalComponents
  • Modelica.Electrical.QuasiStationary.MultiPhase.Blocks.FromSymmetricalComponents
  • Modelica.Electrical.QuasiStationary.MultiPhase.Blocks.SingleToMultiPhase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Critical/severe issue L: Electrical.QuasiStationary Issue addresses Modelica.Electrical.QuasiStationary
Projects
None yet
Development

No branches or pull requests

4 participants