Skip to content

Change the polarity of reactive power#233

Merged
tiyash-basu-frequenz merged 1 commit intofrequenz-floss:v0.x.xfrom
tiyash-basu-frequenz:reverse_reactive_polarity
Apr 5, 2024
Merged

Change the polarity of reactive power#233
tiyash-basu-frequenz merged 1 commit intofrequenz-floss:v0.x.xfrom
tiyash-basu-frequenz:reverse_reactive_polarity

Conversation

@tiyash-basu-frequenz
Copy link
Copy Markdown
Contributor

This commit changes the polarity of reactive power in the protobuf definitions to follow the IEEE 1459-2010 standard definitions. In this standard, positive reactive power is inductive (current is lagging the voltage), and negative reactive power is capacitive (current is leading the voltage).

This commit changes the polarity of reactive power in the protobuf
definitions to follow the IEEE 1459-2010 standard definitions. In this
standard, positive reactive power is inductive (current is lagging the
voltage), and negative reactive power is capacitive (current is leading the
voltage).

Signed-off-by: Tiyash Basu <tiyash.basu@frequenz.com>
@tiyash-basu-frequenz tiyash-basu-frequenz self-assigned this Apr 3, 2024
@tiyash-basu-frequenz tiyash-basu-frequenz requested a review from a team as a code owner April 3, 2024 16:16
@tiyash-basu-frequenz tiyash-basu-frequenz linked an issue Apr 3, 2024 that may be closed by this pull request
@tiyash-basu-frequenz tiyash-basu-frequenz added the type:enhancement New feature or enhancement visitble to users label Apr 3, 2024
@github-actions github-actions Bot added part:docs Affects the documentation part:protobuf Affects the protocol buffer definition files and removed type:enhancement New feature or enhancement visitble to users labels Apr 3, 2024
@tiyash-basu-frequenz tiyash-basu-frequenz added part:docs Affects the documentation and removed part:docs Affects the documentation labels Apr 3, 2024
@tiyash-basu-frequenz
Copy link
Copy Markdown
Contributor Author

@llucax FYI
This was mentioned earlier in frequenz-floss/frequenz-client-microgrid-python#30 (comment)

@llucax llucax added the scope:breaking-change Breaking change, users will need to update their code label Apr 4, 2024
@tiyash-basu-frequenz tiyash-basu-frequenz merged commit e0d7317 into frequenz-floss:v0.x.x Apr 5, 2024
@tiyash-basu-frequenz tiyash-basu-frequenz deleted the reverse_reactive_polarity branch April 5, 2024 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:docs Affects the documentation part:protobuf Affects the protocol buffer definition files scope:breaking-change Breaking change, users will need to update their code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reverse polarity of reactive power

2 participants