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

Fix 'kink' in change in lift due to ground effect tables for C172x an… #3

Merged
merged 1 commit into from Mar 3, 2018

Conversation

seanmcleod
Copy link
Member

…d B737

@bcoconni bcoconni self-assigned this Mar 3, 2018
@bcoconni bcoconni added the bug label Mar 3, 2018
@bcoconni
Copy link
Member

bcoconni commented Mar 3, 2018

Quoting @seanmcleod from the ML

I glanced at a number of other models in the JSBSim distribution and a number of them didn’t have the ground effect modeled at all, and others like the F-15 and F-16 had curves with different data, i.e. they hadn’t been a simple cut-paste of the C172x or B737 model’s data for ground effect.

Thanks Sean.

@bcoconni bcoconni merged commit ad2d515 into JSBSim-Team:master Mar 3, 2018
@bcoconni
Copy link
Member

bcoconni commented Mar 3, 2018

Just for the record, we are using the same correction than the c172p team PR #922

@agodemar
Copy link
Contributor

agodemar commented Mar 4, 2018

Well done.

bcoconni added a commit that referenced this pull request May 16, 2018
Uses GitHub API token instead of Travis'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants