Skip to content

Update modeling documentation structure#128

Merged
pelesh merged 10 commits into
developfrom
birchfield-docs-june25
Jun 6, 2025
Merged

Update modeling documentation structure#128
pelesh merged 10 commits into
developfrom
birchfield-docs-june25

Conversation

@abirchfield
Copy link
Copy Markdown
Collaborator

@abirchfield abirchfield commented Jun 5, 2025

Description

Updates modeling documentation to new structure, copying the GenClassical template developed May 29, 2025.

Proposed changes

Updated the following readme files with the new structure.
Branch
BusFault (new doc)
Load
GENROU
GENSAL

Did not update Bus, Exciter, Governor as these are best addressed in a separate effort.

Checklist

N/A

  • All tests pass.
  • Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • The new code follows GridKit™ style guidelines.
  • There are unit tests for the new code.
  • The new code is documented.
  • The feature branch is rebased with respect to the target branch.

Further comments

None.

@abirchfield abirchfield requested a review from pelesh June 5, 2025 16:22
@abirchfield abirchfield marked this pull request as ready for review June 5, 2025 16:22
@abirchfield abirchfield self-assigned this Jun 5, 2025
@abirchfield abirchfield added the documentation Improvements or additions to documentation label Jun 5, 2025
This was referenced Jun 5, 2025
@abirchfield abirchfield changed the title Birchfield docs june25 Update modeling documentation structure Jun 5, 2025
Copy link
Copy Markdown
Collaborator

@pelesh pelesh left a comment

Choose a reason for hiding this comment

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

Great work!

The one outstanding issue is how we want to define $\omega$. For better clarity, I would suggest defining $\omega$ as the machine speed in [p.u.] rather than relative speed. The relative speed would typically be $\Delta\omega$, but that notation might be cumbersome.

I also made a few nitpicking comments.

Comment thread src/Model/PhasorDynamics/Branch/README.md
Comment thread src/Model/PhasorDynamics/BusFault/README.md
Comment thread src/Model/PhasorDynamics/Load/README.md
Comment thread src/Model/PhasorDynamics/Load/README.md
Comment thread src/Model/PhasorDynamics/SynchronousMachine/GENROUwS/README.md Outdated
Comment thread src/Model/PhasorDynamics/SynchronousMachine/GENSALwS/README.md Outdated
Comment thread src/Model/PhasorDynamics/SynchronousMachine/GENSALwS/README.md Outdated
Comment thread src/Model/PhasorDynamics/SynchronousMachine/GENSALwS/README.md Outdated
Comment thread src/Model/PhasorDynamics/SynchronousMachine/GENROUwS/README.md Outdated
Comment thread src/Model/PhasorDynamics/SynchronousMachine/GENROUwS/README.md Outdated
@abirchfield
Copy link
Copy Markdown
Collaborator Author

OK, made those changes

@abirchfield abirchfield requested a review from pelesh June 6, 2025 01:57
@pelesh pelesh merged commit 593fdbd into develop Jun 6, 2025
@abirchfield abirchfield deleted the birchfield-docs-june25 branch June 6, 2025 14:01
WiktoriaZielinskaORNL pushed a commit that referenced this pull request Jul 23, 2025
* Initial update to GENROU docs

* Initial update to GENSAL docs

* Initial update to load docs

* Initial BusFault docs

* Initial branch docs update

* Correct load math markers

---------

Co-authored-by: abirchfield <abirchfield@users.noreply.github.com>
Co-authored-by: pelesh <peless@ornl.gov>
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

Successfully merging this pull request may close these issues.

2 participants