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

correct get_rate to get_rating #381

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

HaleyRoss
Copy link
Contributor

The branch wrapper hadn't been updated with the new function name which was switched in psy4. Corrected function name.

Copy link
Member

@jd-lara jd-lara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the addition

@jd-lara jd-lara merged commit 895bfd6 into NREL-Sienna:jd/psy4_update Jul 2, 2024
jd-lara added a commit that referenced this pull request Jul 8, 2024
* correct get_rate to get_rating (#381)

* bump dependencies

* Hr/add limiters (#376)

* Add current limiters to inner_conrol_models

* Fix limiter integration mistakes

* Add saturation limiting logic

* Add hybrid limiter control logic

* Correct sign error hybrid limiter

* Remove print

* Change PowerSystems version

* modify inner control gain

* debug

* fix mag limiter

* change placement of kiv in hyblim

* debug load

* revert debug

---------

Co-authored-by: Rodrigo Henríquez-Auba <rodrigomha@gmail.com>

* bump project toml

* formatter

* reuse multiple dispatch

* change exponential names

* update json exponential coefficients

* revert IS check

* change Union to real types

* try catch for test 10

* add sim type

* update printing

* change to logs

* update printing

* add directly the LinearMatrix

* revert try and catch

---------

Co-authored-by: HaleyRoss <125164295+HaleyRoss@users.noreply.github.com>
Co-authored-by: Rodrigo Henríquez-Auba <rodrigomha@gmail.com>
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 this pull request may close these issues.

2 participants