Skip to content

Fix logging for valid bustype and devices#1434

Merged
jd-lara merged 2 commits into
psy5from
mb/fix-bus-parsing-log
Jun 25, 2025
Merged

Fix logging for valid bustype and devices#1434
jd-lara merged 2 commits into
psy5from
mb/fix-bus-parsing-log

Conversation

@m-bossart

Copy link
Copy Markdown
Contributor

Fix the logic of proper combinations of bustypes and devices:

  • Checks that PV and SLACK buses have either a non-electric load static injection or an hvdc connected.

  • Does not check PQ buses.

Fixes excessive logging in tests to to ACTIVG systems. If the underlying data for these systems is corrected, the tests can be updated.

@m-bossart m-bossart requested a review from rodrigomha June 25, 2025 18:55
@jd-lara jd-lara self-requested a review June 25, 2025 20:02
if !get_available(gen)
bus_to_pv_device_map = Dict((i, Set{String}()) for i in get_number.(buses))
for gen in get_components(StaticInjection, sys)
if !get_available(gen) || typeof(gen) == ElectricLoad

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This makes me think we need a trait for static injectors as something like supports_voltage_control because we have a similar pattern in PowerFlows.jl

@jd-lara jd-lara self-assigned this Jun 25, 2025
@jd-lara jd-lara merged commit 4f3a0ee into psy5 Jun 25, 2025
1 of 9 checks passed
@jd-lara jd-lara deleted the mb/fix-bus-parsing-log branch August 1, 2025 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants