Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Updating the pole variables to be public #939

Merged
merged 2 commits into from
Jul 2, 2021
Merged

Updating the pole variables to be public #939

merged 2 commits into from
Jul 2, 2021

Conversation

aivanova5
Copy link
Collaborator

This PR fixes issue lack of access to some pole variables desired to have on output.

Current issues

None

Code changes

  • module/powerflow/pole.cpp

Documentation changes

None

Test and Validation Notes

None (the usual)

@aivanova5 aivanova5 requested a review from dchassin July 1, 2021 21:17
@aivanova5 aivanova5 changed the base branch from master to develop July 1, 2021 21:19
dchassin
dchassin suggested changes Jul 1, 2021
View reviewed changes
Copy link
Member

@dchassin dchassin left a comment

Choose a reason for hiding this comment

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

I don't think these the moment at pole failure. Rather they're moment at current wind speed.

@dchassin dchassin added the bug Request a fix or change to the code label Jul 1, 2021
@aivanova5 aivanova5 requested a review from dchassin July 1, 2021 22:36
@dchassin dchassin merged commit 9604527 into develop Jul 2, 2021
@dchassin dchassin deleted the develop-publish-pole-vars branch July 2, 2021 00:36
@dchassin dchassin mentioned this pull request Jul 2, 2021
Merged
11 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dchassin dchassin dchassin approved these changes

Assignees
No one assigned
Labels
bug Request a fix or change to the code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aivanova5 @dchassin