Skip to content
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

Further improvements and bug fixes to FAST.Farm case generation #15

Merged
merged 17 commits into from
Aug 29, 2024

Conversation

rthedin
Copy link
Collaborator

@rthedin rthedin commented Jun 11, 2024

Main changes include:

Other smaller changes:

  • More descriptive docstrings on some functions
  • Longer error messages with more details
  • Better error/warning errors
  • Smaller fixes and improvements

This is a draft PR as I will continue to work on it. Submitting as a draft for visibility.

- Adding ability to give what level the box will be in. This is relevant
for low-res boxes that will not be in level 0
- More robust error/warning printing
- Several improvements
Plus additional smaller fixes and improvements
@rthedin rthedin added the enhancement New feature or request label Jun 11, 2024
@rthedin rthedin marked this pull request as ready for review August 27, 2024 23:27
@rthedin
Copy link
Collaborator Author

rthedin commented Aug 27, 2024

I've made some progress in fixing issues and making general improvements to the toolbox when it comes to FAST.Farm case setup. I think we should merge this in and then open other similar PRs with more fixes/features. I don't want this PR to become huge.

@rthedin rthedin merged commit cfb3f1d into OpenFAST:main Aug 29, 2024
2 of 7 checks passed
@rthedin rthedin deleted the f/ff_improvements branch August 29, 2024 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants