-
Notifications
You must be signed in to change notification settings - Fork 179
Description
Notebooks in the documentation have failing code cells but no error was raised in GH Actions
A couple of Jupyter Notebooks used to create the online documentation have failing code cells:
-
Background and Concepts - The Background and Concepts section refers to a variable
grid.x
that was removed in Add coordinates from inertial frame to grid class #578 in favor ofgrid.x_sorted
. This notebook should be updated accordingly, and documentation should be included to explain the_sorted
variables, in general (see Repeated calls to calculate wake can result in wrong wake calculations #378). -
Turbine Library - The Turbine Library page is breaking due to a relative bath being hard-coded in the turbine definition. See Multidimensional Cp and Ct IEA 15MW definition can fail to load #737 for details.
More problematic is that GitHub Actions did not end with a failure when these breaking changes were introduced, so we were made aware much later.
The follow sequential steps are required to resolve this issue:
- 1 GitHub Actions should produce a failing result with the documentation in its current state
- Resolved in Catch documentation and examples errors #736
- 2 Change the Getting Started notebook to refer to
grid.x_sorted
- 3 Describe the meaning and relationships of the following variables:
grid.x
,grid.x_sorted
,grid.x_sorted_inertial_frame
- 4 Address Multidimensional Cp and Ct IEA 15MW definition can fail to load #737 to support loading the mutlidimensional Cp and Ct definition for IEA 15MW from any path outside the
floris/examples/
directory
How to reproduce
The broken code cells can be seen in the live docs:
Floris version
The documentation builds from the develop
branch.