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.

Sign up for GitHub

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

Jump to bottom

add field "converged" to PowerFlowData, return NaN when not converged #82

Merged
merged 5 commits into from
Jan 21, 2025
Merged

add field "converged" to PowerFlowData, return NaN when not converged #82

merged 5 commits into from
Jan 21, 2025

Conversation

rbolgaryn
Copy link

  • add converged field to PowerFlowdata
  • NaN for not converged time steps

@rbolgaryn rbolgaryn added the enhancement New feature or request label Jan 16, 2025
@rbolgaryn rbolgaryn requested a review from GabrielKS January 16, 2025 22:17
@GabrielKS GabrielKS mentioned this pull request Jan 21, 2025
Merged
@GabrielKS
Copy link
Contributor

Merging now to keep the branches from diverging too much. TODO add docstrings and tests.

@GabrielKS GabrielKS merged commit 5a1f97b into hrgks/psse_exporter_psy4 Jan 21, 2025
3 of 6 checks passed
@GabrielKS GabrielKS deleted the feature/nr_pf2 branch January 21, 2025 18:59
@GabrielKS GabrielKS mentioned this pull request Jan 21, 2025
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GabrielKS GabrielKS Awaiting requested review from GabrielKS

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rbolgaryn @GabrielKS