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

Line2-to-Point Mapping Search Bug #8

Closed
jjonkman opened this issue Mar 3, 2017 · 2 comments
Closed

Line2-to-Point Mapping Search Bug #8

jjonkman opened this issue Mar 3, 2017 · 2 comments
Assignees
@HaymanConsulting
Labels
Type: Bug

Comments

@jjonkman
Copy link
Collaborator

jjonkman commented Mar 3, 2017

The Line2-to-Point mesh-mapping algorithm for loads documented in our AIAA paper is correct. However, there is a bug in the implementation of the mapping search part of the algorithm that leads to the problems--basically, while the total integrated loads are correct, the local loads distributed along the Point elements are incorrect when the number of points elements is greater than the number of Line2 nodes.

FAST users have reported this problem in the following forum posts:
https://wind.nrel.gov/forum/wind/viewtopic.php?f=4&t=1767
https://wind.nrel.gov/forum/wind/viewtopic.php?f=3&t=1701
@jjonkman
Copy link
Collaborator Author

Bonnie Jonkman of Envision fixed the bug! See the commit here: old-NWTC/NWTC_Library@ba1bed6. The fix can be easily merged into the OpenFAST repo.

@michaelasprague
Copy link
Contributor

@ghaymanNREL , would you please merge this fix? Thanks!

@evan-gaertner evan-gaertner mentioned this issue Oct 18, 2018
Closed
@evan-gaertner evan-gaertner mentioned this issue Jan 3, 2019
Closed
@ghost ghost mentioned this issue Apr 16, 2019
Closed
@Pouria-AZD Pouria-AZD mentioned this issue Jun 18, 2019
Closed
@CallumG-Ed CallumG-Ed mentioned this issue Jan 24, 2020
Closed
@caroledaniel caroledaniel mentioned this issue Feb 27, 2020
Closed
rdamiani pushed a commit to rdamiani/openfast that referenced this issue Mar 10, 2020
@bjonkman
Merge pull request OpenFAST#8 from ebranlard/f/Linear
b325244 
Update of VS solution
andrew-platt referenced this issue in andrew-platt/openfast Mar 19, 2020
@ebranlard
Merge pull request #8 from ptrbortolotti/noise
4c9ce56 
bug fixing in the TI noise model implementation
@AtilaOnaya AtilaOnaya mentioned this issue Apr 10, 2020
Closed
andrew-platt pushed a commit that referenced this issue Jul 31, 2020
@ptrbortolotti
Merge pull request #8 from rafmudaf/noise
dd258fc 
Adds regression test results for Intel compiler
@hamidMech hamidMech mentioned this issue Nov 11, 2020
Closed
mattEhall referenced this issue in mattEhall/openfast Dec 3, 2020
@jjonkman
Merge pull request #8 from ebranlard/f/fast-farm
fa190a1 
adding OPENMP option flag for cmake
@Seager1989 Seager1989 mentioned this issue Jan 11, 2021
Closed
@Christophe-delFosse Christophe-delFosse mentioned this issue Apr 8, 2021
Closed
@hkross hkross mentioned this issue Dec 20, 2021
Open
andrew-platt pushed a commit that referenced this issue May 31, 2023
@luwang00
Merge pull request #8 from bjonkman/f/Hydro_SeaState_PR1008
be08721 
Merge OpenFAST dev + other SeaState/HydroDyn code cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HaymanConsulting HaymanConsulting

Labels
Type: Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@michaelasprague @jjonkman @HaymanConsulting