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

Contribution #8

Closed
lhbueno opened this issue Feb 17, 2017 · 5 comments
Closed

Contribution #8

lhbueno opened this issue Feb 17, 2017 · 5 comments

Comments

@lhbueno
Copy link

lhbueno commented Feb 17, 2017

How could I contribute to this library?
I developed a block to work with Modelica Electrical MultiPhase library as you can see in the figure below.
It has also reactive power control.

image

@lhbueno lhbueno closed this as completed Feb 17, 2017
@lhbueno lhbueno changed the title sdfsdffds A Feb 17, 2017
@lhbueno lhbueno changed the title A Contribution Feb 17, 2017
@lhbueno lhbueno reopened this Feb 17, 2017
@christiankral
Copy link
Owner

You are very welcome to contribute. Please fork the library and make a pull request or just send me your model.

@lhbueno
Copy link
Author

lhbueno commented Feb 20, 2017

The idea of this improvement is to interface the PV system to the Power System library

image

@christiankral
Copy link
Owner

Manually incorporated your pull request in 92df23f. The following changes have additionally been performed:

In model MultiPhaseConverter

  • Blocks.Rotation_dq is replaced by Modelica.Electrical.Machines.SpacePhasors.Blocks.Rotator
  • Blocks.Park is replaced by Modelica.Electrical.Machines.SpacePhasors.Blocks.ToSpacePhasor toSpacePhasor and Modelica.Electrical.Machines.SpacePhasors.Blocks.Rotator
  • fromSpacePhasor is not diectly connected to the signal current source; instead a smoothing first order delay is used; this enables two things:
    • The resistors parallel to the voltages sources in PhotoVoltaics.Examples.SimpleModuleMultiPhase is not required any more
    • An ideal inductor between converter and voltage source does not cause the simulation model to fail

The following models are removed due to redundancy reasons

  • Components.Blocks.MIMO
  • Components.Blocks.Park
  • Components.Blocks.Rotation_dq
  • Components.Blocks.Icons

@christiankral
Copy link
Owner

See #11

@christiankral
Copy link
Owner

Model ist re-structured and updated according to #12

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

No branches or pull requests

2 participants