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

Undefined variables in function calc_theta_delta_bounds #870

Closed
rb004f opened this issue Aug 10, 2023 · 2 comments · Fixed by #871
Closed

Undefined variables in function calc_theta_delta_bounds #870

rb004f opened this issue Aug 10, 2023 · 2 comments · Fixed by #871

Comments

@rb004f
Copy link
Collaborator

rb004f commented Aug 10, 2023

In function calc_theta_delta_bounds, when the function drops into line 76, it references undefined variables amin and amax and throws an UndefVarError

@ccoffrin
Copy link
Member

Good catch. I have merged a test and a fix, do you need a new release tagged right away?

On a side note, this example made me realize there is a bunch of "conductor" based features that could probably be removed from PM now that PMD is basically a stand alone package. Do you see a reason to keep supporting "conductors" in PM?

CC @pseudocubic

@rb004f
Copy link
Collaborator Author

rb004f commented Aug 11, 2023

No, I don't need a new release right away. And yes, it seems like we are migrating away from having multi conductor support in PowerModels and have it all in PMD.

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 a pull request may close this issue.

2 participants