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.
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
Remove unused code from floris.tools and add tool reg tests #819
Conversation
b1a8344
to
c6de7fe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just double checking that we want to remove this from the examples since, currently, this function as the user documentation. If you're planning to add it back, is this captured somewhere so that it doesn't fall off the list of things to document?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My understanding is that the new wind data functionality replaces this functionality, and is covered in example 34. When the examples are processed, that would be a good time to bring it back into the documentation.
d04572f
to
9a7f3bb
Compare
Clean up tools and add tool reg tests
This PR removes unused code from the
floris.tools
directory, and adds some simple reg tests for the yaw and layout optimization tools.Related issue
None.
Impacted areas of the software
floris.tools
tests/reg_tests
Additional supporting information
None.
Test results, if applicable
Passing.