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 inverter models with current limiting #328

Closed
m-bossart opened this issue May 16, 2023 · 3 comments
Closed

Add inverter models with current limiting #328

m-bossart opened this issue May 16, 2023 · 3 comments
Assignees

Comments

@m-bossart
Copy link
Collaborator

The existing inverter models do not include current limiting. It would be good to include some models from the literature that have current limiting. @ciaranrob has started an implementation.

One key question is how to make the addition of current limiting modular and extensible? Does it make sense to define a new model (e.g. CurrentLimitedVoltageModeControl) which is essentially equivalent to VoltageModeControl with the addition of an Imax parameter or is there a better way? @jd-lara @rodrigomha do you have thoughts on this?

@m-bossart m-bossart self-assigned this May 16, 2023
@rodrigomha
Copy link
Contributor

That proposed option is okay for now, but it is not reasonable to do that more than once, so if we want to try other limiters we need a long-term solution.

I think the proper solution requires some major changes.

An intermediate solution would be similar of how the outer control work: create a struct for InnerControl that contains the VoltageControl, CurrentLimiter and CurrentControl.

Then, you can probably dispatch the combinations that you want in a clever way (using _mdl_ode_functions to dispatch internally any combination you want) or repeat code.

This would require to do a bunch of things:

  • Define a NoCurrentLimiter struct
  • Define a NoVoltageControl struct
  • Split VoltageModeControl into something like VoltageControlVImpedance, NoCurrentLimiter and StandardCurrentControl
  • Split CurrentModeControl into something like NoVoltageControl, NoCurrentLimiter and StandardCurrentControl

What do you think @jd-lara ?

@jd-lara
Copy link
Member

jd-lara commented Jul 7, 2023

looking into this issue it seems we need to modify the structures of the inverters to incorporate current limiter DynamicComponents. We need to PoC this for PSY 3.0.

@rodrigomha
Copy link
Contributor

This closed in #383

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

3 participants