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

Fix NetworkSwitch perturbation tutorial #342

Open
rodrigomha opened this issue Jul 10, 2023 · 1 comment
Open

Fix NetworkSwitch perturbation tutorial #342

rodrigomha opened this issue Jul 10, 2023 · 1 comment
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@rodrigomha
Copy link
Contributor

rodrigomha commented Jul 10, 2023

We need to explicitly state to use using PowerNetworkMatrices when creating Ybus.

In addition, we should discuss if NetworkSwitch should ONLY use a SparseMatrix or not. From a user perspective we should allow the user to use directly a dense matrix if they want.

In any case, we need to update the tutorial of Perturbations since it is using Dense Matrices: https://nrel-sienna.github.io/PowerSimulationsDynamics.jl/stable/perturbations/

image

@rodrigomha rodrigomha added bug Something isn't working documentation Improvements or additions to documentation labels Jul 10, 2023
@jd-lara jd-lara removed the bug Something isn't working label Jul 11, 2023
@jd-lara
Copy link
Member

jd-lara commented Jul 11, 2023

I don't think this is a bug. We have always assumed that the user would pass a Bus object which is by default sparse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants