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 phasor dynamics bus and branch models #38

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

Conversation

pelesh
Copy link
Collaborator

@pelesh pelesh commented Dec 23, 2024

Models for a bus and a branch for phasor dynamics simulations are added. These models use current balance and Cartesian coordinates.

Add bus component model.
Corrected paths to figures in README files for phasor dynamics component models.
@pelesh pelesh requested a review from abirchfield December 23, 2024 22:15
@pelesh pelesh self-assigned this Dec 23, 2024
@pelesh pelesh marked this pull request as draft December 23, 2024 22:15

private:
// Default initial values for voltage and phase on PQ bus
ScalarT Va0_{0.0};
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be Vr0 and Vi0, along with the constructor.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in c79256c.

using BusData = GridKit::PowerSystemData::BusData<real_type, IdxT>;

Bus();
Bus(ScalarT Va, ScalarT Vr);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Vr, Vi

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in c79256c.

*/
template <class ScalarT, typename IdxT>
Bus<ScalarT, IdxT>::Bus()
: Va0_(0.0), Vr0_(0.0)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Vr,Vi

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in c79256c.

*/
template <class ScalarT, typename IdxT>
Bus<ScalarT, IdxT>::Bus(ScalarT Va, ScalarT Vr)
: Va0_(Va), Vr0_(Vr)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Vr,Vi

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in c79256c.

*/
template <class ScalarT, typename IdxT>
Bus<ScalarT, IdxT>::Bus(BusData& data)
: Va0_(data.Vm * cos(data.Va)), Vr0_(data.Vm * sin(data.Va))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Vr,Vi


A bus is a point of interconnection of electrical devices. Bus component model
also plays a key role in coupling system components. Each bus $`i`$ owns two
variables -- active and reactive voltage $`V_{ai}`$ and $`V_{ri}`$,
Copy link
Collaborator

Choose a reason for hiding this comment

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

The terms "active and reactive voltage" have no physical meaning. We should say "real and imaginary components of the voltage phasor". Same thing with current. On the other hand, active and reactive power are OK to talk about.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in c79256c.

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