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

Improve User-Specified Wave-Kinematics File Formatting Requirements for WaveMod = 6 #118

Closed
jjonkman opened this issue May 17, 2018 · 3 comments · Fixed by #357
Closed

Comments

@jjonkman
Copy link
Collaborator

The data format of the various user-specified wave-kinematics input files (.Elev, .Vxi, .Vyi, .Vzi, .Axi, .Ayi, .Azi, and .DynP) used with the WaveMod = 6 option in HydroDyn are very finicky, with very specific requirements regarding data format, white space, etc. This has caused many users trouble. HydroDyn should be made more robust so that the data format requirements are more user-friendly. More information can be found in the following forum topic: https://wind.nrel.gov/forum/wind/viewtopic.php?f=4&t=1690.

@ebranlard
Copy link
Contributor

ebranlard commented Feb 7, 2019

I've added some checks for this format (see merge request #239 above). For now, the file is allowed to have more lines and more columns than required, but not less (before it could). I can add test to check for strict equality of number of lines and columns if needed.

rafmudaf added a commit that referenced this issue Mar 27, 2019
Adding safety on number of lines and number of fields per line when WaveMode=6 (Should Fix #118)
@rafmudaf
Copy link
Collaborator

rafmudaf commented Mar 27, 2019

fixed by #239 maybe not quite fixed?

rafmudaf pushed a commit to rafmudaf/openfast that referenced this issue Apr 8, 2019
rafmudaf pushed a commit to rafmudaf/openfast that referenced this issue Apr 9, 2019
@SatishJ013
Copy link

Hi all,

I am facing issue with externally generated wave kinematics WaveMod=6 in OpenFAST-v2.4.0 as I am getting wave elevations of zero in output file. I have attached the relevant input files. Kindly suggest me on this.

Thanks,
Satish J
TEST_004 (2).zip
TEST_004.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants