Skip to content

Migrate the HydroDyn Manual to readthedocs#753

Merged
rafmudaf merged 21 commits intoOpenFAST:devfrom
rafmudaf:docs/hydrodyn_manual
Aug 6, 2021
Merged

Migrate the HydroDyn Manual to readthedocs#753
rafmudaf merged 21 commits intoOpenFAST:devfrom
rafmudaf:docs/hydrodyn_manual

Conversation

@rafmudaf
Copy link
Collaborator

Feature or improvement description
This pull request migrates the HydroDyn Manual from FAST 8 into the OpenFAST web-based documentation on readthedocs.
See it live at https://raf-openfast.readthedocs.io/en/docs-hydrodyn_manual/source/user/hydrodyn/index.html

Related issue, if one exists
#179

Impacted areas of the software
HydroDyn documentation

@rafmudaf
Copy link
Collaborator Author

rafmudaf commented Jun 2, 2021

For reference: rafmudaf#16

@rafmudaf
Copy link
Collaborator Author

rafmudaf commented Aug 5, 2021

I've migrated the remaining missing equations that I and @andrew-platt have found. I believe this pull request is now ready to go! @jjonkman @andrew-platt do either of you want to take another look?

https://raf-openfast.readthedocs.io/en/docs-hydrodyn_manual/source/user/hydrodyn/index.html

@jjonkman
Copy link
Collaborator

jjonkman commented Aug 5, 2021

I would like to review this at some point and make any changes necessary based on the recent updates to HydroDyn in v2.6. But I probably won't get to this for a couple weeks and I don't want to hold up the merge either, so, I'm happy if you want to merge it now and I'll review at a later point.

@rafmudaf
Copy link
Collaborator Author

rafmudaf commented Aug 5, 2021

@jjonkman That sounds good to me. At this point, this pull request is specific to migrating the documentation to readthedocs so we can have another one for updating the HydroDyn documentation to reflect the latest changes.

@rafmudaf rafmudaf merged commit ef79e5d into OpenFAST:dev Aug 6, 2021
@rafmudaf rafmudaf deleted the docs/hydrodyn_manual branch August 31, 2021 21:35
@rafmudaf rafmudaf mentioned this pull request Mar 2, 2022
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments