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

Switch to openfast_io and update to OpenFAST v4.0.0 #415

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

mayankchetan
Copy link
Contributor

@mayankchetan mayankchetan commented Jan 12, 2025

Description and Purpose

This PR updates ROSCO to use the new openfast_io library for reading and writing OpenFAST files. Additionally, ROSCO is updated to track OpenFAST v4.0.0

Dependancies for PR:

WEIS PR (WISDEM/WEIS#341) depends on this PR

Type of change

What types of change is it?
Select the appropriate type(s) that describe this PR

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (non-backwards-compatible fix or feature)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Documentation update
  • Maintenance update
  • Other (please describe): Testing and examples

TODO Items General:

  • Update examples & testing based on @dzalkind's recommendations
  • ROSCO PR () & openfast_io PR () need to be merged

TODO Items API Change:

  • Update docs with API change
  • Run update_rosco_discons.py in Test_Cases/
  • Update DISCON schema

Github issues addressed, if one exists

Examples/Testing, if applicable

@mayankchetan mayankchetan changed the base branch from main to develop January 12, 2025 07:11
@dzalkind dzalkind mentioned this pull request Jan 22, 2025
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants