Replies: 1 comment 3 replies
-
The documentation was migrated to Jupyter-book from Jekyll and content expanded in #514. That pull request was meant to mostly move to the new framework, but @RHammond2 gave good feedback on the content overall, so I've copied it here for reference. FLORIS Intro
Installation
Developer's Guide
Background and ConceptsIntro
Get turbine power
Visualization
API
Code Quality
|
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
There will be two main perspectives in developing the documentation: user-facing and developer-facing. Here, let's outline ideas and considerations for both. This is closely related to the examples discussion.
Reference
User documentation
Describe important concepts:
!include <path>
to point to your own turbine definitionDescription of turbine model definition
How to get wind speeds at each turbine (#422).
Describe the turbine models that are included. Link to https://nrel.github.io/turbine-models/index.html.
Using the public API (i.e.
FlorisInterface.reinitialize
and etc)fi.reinitialize
can take multiple arguments or you can call it multiple times. It continuously modifies the same data unless you reinstantiate.Developer documentation
Describe how all the classes are created using attrs from a dictionary
field()
Generally, most input validation should occur in the
FlorisInterface
. This is where the "business logic" exists, andfloris.simulation
is maintained as a generic, parameterized simulation framework.Implementation Details
Describe the meaning of the dimensions of the arrays.
Describe the
_sorted
vs unsorted variables.FlowField.u,v,w
andGrid.x_sorted,y_sorted,z_sorted
. The description of these variables should either address why they're inconsistently named or make their names consistent.Describe inertial coordinates (vs wind-relative or rotated).
Describe how the turbulence intensity is set and expanded throughout the calculation. Specifically, see Bug fix: sort turbulence intensity field in post-calculation step #564 and Bug fix: initialize TI array with correct dimensions #584.
Beta Was this translation helpful? Give feedback.
All reactions