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

new zip coefficents for loads #2391

Draft
wants to merge 12 commits into
base: develop
Choose a base branch
from

Conversation

pawellytaev
Copy link
Contributor

In pandapower the zip coefficients const_i_percent and const_z_percent were defined both for the P and the Q voltage dependency of loads. This PR separates them and introduces const_i_p_percent, const_i_q_percent, const_z_p_percent, const_z_q_percent to allow separate P and Q voltage dependecies in the power flow. const_i_percent and const_z_percent are replaced with these new columns.

@quant12345
Copy link
Contributor

quant12345 commented Sep 9, 2024

I see that you also have a build (3.11, 1) failure. Here I described the reason for the test failure: test_from_mpc.py. It is not clear why other builds pass tests?

@pawellytaev
Copy link
Contributor Author

pawellytaev commented Sep 9, 2024

yeah, looks strange to me also. We made some changes to the mpc and ppc converters recently in PR #2347, but this bug didn't appear back then... Will let you know if we find out something

@quant12345
Copy link
Contributor

quant12345 commented Sep 9, 2024

I had a local test that was skipped because of the decorator (as far as I understand, if matpowercaseframes is not installed, the test is skipped), so there was no error.
Anyway, in issues I suggested two options for fixing it. Let me know if a fix is ​​needed?

And I also think we need to look at when the migration to numpy 2.0 took place. Perhaps earlier versions weren't so strict.

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.

3 participants