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 trafo/shunt characteristics and phase shifter logics #2490

Open
wants to merge 134 commits into
base: develop
Choose a base branch
from

Conversation

panos-xenos
Copy link

@panos-xenos panos-xenos commented Dec 12, 2024

  • CGMES2pp converter: add trafo_characteristic_table & shunt__characteristic_table; amend trafo spline characteristic logic to use characteristic table; introduce shunt spline characteristics; amend/rename/introduce trafo/trafo3w/shunt parameters to reflect changes.
  • Phase shifters: introduce different types of phase shifters for trafo and trafo3w; amend build_branch to reflect all changes.
  • Pytests: amend/introduce pytests & corresponding libs/test network files to reflect changes above.

panos-xenos and others added 30 commits October 9, 2024 18:39
…s into the converter; move trafo & shunt spline characteristic creation out of the converter (old code commented)
…dex with id_characteristic_table; remove individual spline characteristic indices from trafo and shunt tables; create one row per trafo/shunt in trafo_characteristic_spline & shunt_characteristic_spline tables with all spline characteristic objects
…p_step_degree with the values corresponding to the neutral tap position for PhaseTapChangerTabular trafos
# Conflicts:
#	pandapower/test/api/test_auxiliary.py
tfabian and others added 27 commits December 20, 2024 10:03
…cs_revertJson

revert json files back (because current status was only half-updated and wrong)
…cs_deleteCharacteristic

delete "characteristic" from json file
…cteristics_merge_trafocharactersitic

# Conflicts:
#	pandapower/contingency/contingency.py
…cs_merge_trafocharactersitic

 merge trafo charactersitic
@panos-xenos panos-xenos marked this pull request as ready for review December 20, 2024 18:07
@panos-xenos
Copy link
Author

@panos-xenos, @hilbrich, @Fabsit could you check the broken tests and fix them?

@vogt31337 all required tests and pipelines are now passing, please consider this PR for pre-release 3.0 - many thanks!

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.

8 participants