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.

Sign up for GitHub

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

Jump to bottom

Controls Applications Cases #30

Merged
merged 10 commits into from
May 3, 2023
Merged

Controls Applications Cases #30

merged 10 commits into from
May 3, 2023

Conversation

jtgrasb
Copy link
Contributor

@jtgrasb jtgrasb commented Mar 24, 2023

This PR is a rework of PR #967 from the WEC-Sim master repo. This includes a folder of controller application cases including passive, reactive, latching, declutching, and model predictive controllers.

@jtgrasb jtgrasb mentioned this pull request Mar 24, 2023
Merged
@nathanmtom nathanmtom requested a review from jleonqu April 5, 2023 14:28
@jleonqu
Copy link
Contributor

jleonqu commented Apr 25, 2023

Hi @jtgrasb
Thanks for putting together all the controller examples. This will be really useful for WEC-Sim users. All the examples are good references for the different types of controllers that you have included in the documentation. This PR is ready to merge into the main repository. I am not sure if the tests that are not passing will cause problems when this is merged, I run all the examples locally and I didn't have any issues. I was using Matlab 2022b.

@jleonqu jleonqu merged commit 33b3f69 into WEC-Sim:dev May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jleonqu jleonqu jleonqu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jtgrasb @jleonqu