Skip to content
This repository has been archived by the owner on Jul 17, 2020. It is now read-only.

[WIP] Last updates #67

Merged
merged 21 commits into from
Jul 16, 2020
Merged

[WIP] Last updates #67

merged 21 commits into from
Jul 16, 2020

Conversation

rodrigomha
Copy link
Member

@rodrigomha rodrigomha commented Jul 16, 2020

The following PR updates Test 10 and 11 to properly assess the effects of Dynamic Lines.
It also adds an update model for Source, that now uses an internal voltage and angle to match the solution of the power flow (via a new initialization function). Finally, new functions to print initial conditions are provided. The new printer provides bus voltages, states of dynamic devices and current for dynamic lines.

To do:

  • Initialization for Dynamic Lines
  • Update documentation example for Dynamic Lines

@rodrigomha rodrigomha requested a review from jd-lara July 16, 2020 06:26
@jd-lara
Copy link
Member

jd-lara commented Jul 16, 2020

@rodrigomha this will also require the serialization of DynamicLines.

@jd-lara jd-lara self-assigned this Jul 16, 2020
@rodrigomha
Copy link
Member Author

rodrigomha commented Jul 16, 2020

@rodrigomha this will also require the serialization of DynamicLines.

Yes. We need that to use the deepcopy of the system. We also need to update the versions of the project. I didn't update the project here, but tests are passing with PSY#master.

src/utils/plot_utils.jl Outdated Show resolved Hide resolved
src/utils/plot_utils.jl Outdated Show resolved Hide resolved
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.

There are several things in the result that retrieve code that is not necessary. Please see the comments, we already have a sorted dict with the lookup in the simulation.

@jd-lara jd-lara merged commit b5f6244 into jd/handle_dyn_brancehs Jul 16, 2020
@jd-lara jd-lara deleted the rh/final_updates branch July 16, 2020 19:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants