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.

Sign up for GitHub

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

Jump to bottom

BeamDyn initialization with non-zero initial azimuth produces strange results #26

Closed
bjonkman opened this issue May 10, 2017 · 5 comments
Closed

BeamDyn initialization with non-zero initial azimuth produces strange results #26

bjonkman opened this issue May 10, 2017 · 5 comments
Assignees
@rafmudaf
Labels
Module: BeamDyn Type: Bug

Comments

@bjonkman
Copy link
Contributor

See attached file for description of the problem.

BeamDyn_InitializationBug.pdf
@bjonkman
Copy link
Contributor Author

bjonkman commented Aug 1, 2017

@michaelasprague - Has there been any progress on this issue?

@michaelasprague michaelasprague assigned ghost Aug 1, 2017
@michaelasprague
Copy link
Contributor

@bjonkman , sorry, no. @rafaelmudafort is taking lead on this now; plan is to address this by exposing the problem in the testing harness and to then pursue the fix.

@andrew-platt
Copy link
Collaborator

andrew-platt commented Aug 10, 2017 •
edited
Loading

In the CalcIC routine, the calculation of the translational velocity is incorrectly referenced. The root node position needs to be subtracted otherwise the distance used in the cross product is relative to the original root position, not the rotated root position. We (Envision) have a fix for this that we will give you.

BTW: the fix we have for this appears to help get a larger timestep.

@michaelasprague
Copy link
Contributor

@andrew-platt, it would be great if this could be submitted as a pull request -- please let us know if there's another mechanism you'd prefer. Thanks!

bjonkman added a commit to bjonkman/openfast that referenced this issue Aug 14, 2017
@bjonkman
Fix for issue OpenFAST#26: BeamDyn initialization with non-zero initi…
97d1c2a 
…al azimuth

The initial translation velocity calculation didn't remove the location of the root node when taking the cross product with the rotational velocity. This fixes OpenFAST#26
@rafmudaf rafmudaf assigned rafmudaf and unassigned ghost Nov 9, 2017
@rafmudaf rafmudaf mentioned this issue Apr 20, 2018
Closed
@rafmudaf rafmudaf mentioned this issue May 10, 2018
Merged
@rafmudaf
Copy link
Collaborator

This issue was fixed in pull request #114

@ghost ghost mentioned this issue Apr 16, 2019
Closed
@CallumG-Ed CallumG-Ed mentioned this issue Jan 24, 2020
Closed
@caroledaniel caroledaniel mentioned this issue Feb 27, 2020
Closed
@AtilaOnaya AtilaOnaya mentioned this issue Apr 10, 2020
Closed
@caroledaniel caroledaniel mentioned this issue Jun 25, 2020
Closed
This was referenced Nov 29, 2020
Closed
Closed
@Christophe-delFosse Christophe-delFosse mentioned this issue Apr 8, 2021
Closed
@KatrijnChys KatrijnChys mentioned this issue Apr 28, 2021
Closed
@hamidMech hamidMech mentioned this issue Nov 3, 2021
Merged
@yuangao0916 yuangao0916 mentioned this issue May 24, 2022
Open
@LaurenceWETI LaurenceWETI mentioned this issue Sep 20, 2022
Open
psakievich pushed a commit to psakievich/openfast that referenced this issue Oct 26, 2023
@gantech
Merge pull request OpenFAST#26 from jrood-nrel/jrood/fix-fpe
c2aea80 
Check if restart frequency is > 0 to avoid dividing by 0
@bweiss222 bweiss222 mentioned this issue Oct 13, 2024
Open
andrew-platt added a commit that referenced this issue Nov 8, 2024
@andrew-platt
Merge pull request #26 from mayankchetan/b/docker_build
b9c7df1 
repository name must be lowercase
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rafmudaf rafmudaf

Labels
Module: BeamDyn Type: Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@michaelasprague @andrew-platt @bjonkman @rafmudaf