Closed misi9170 closed 2 months ago
Looks really nice @misi9170 , went through and left a few comments. Also I tested tests and examples and ran well
Since there are a few separate pieces that will need to go into a new class to (eventually) replace the ParallelFlorisModel
, I've created a new branch on NREL/floris and have changed the base to point to that branch. There may be a handful of PRs to that branch as we develop it (which will be merged with "squash and merge" as they go in), and finally a merge commit to merge parallel-model-rework into develop when we are happy with the replacement class.
Attempt to make a
ParallelFlorisModel
that is completely consistent with theFlorisModel
introduced in v4.Work in progress. Tests currently passing, however, more tests are needed.