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

Aircraft approach & autoland causes plane crash #1312

Closed
morphelianus opened this issue Jul 7, 2020 · 6 comments · Fixed by #1318
Closed

Aircraft approach & autoland causes plane crash #1312

morphelianus opened this issue Jul 7, 2020 · 6 comments · Fixed by #1318

Comments

@morphelianus
Copy link

morphelianus commented Jul 7, 2020

MechJeb Version

2.10.0.0

KSP Version

Stock 1.10.0.2917 de-de
Making History 1.10.0
Breaking Ground 1.5.0

Description

Whenever I enable the Aircraft Approach & Autoland the planes crashes instandly as MechJeb sets the Target Altitude to -212608 meter. This bug occurs since MechJeb 2.10.0.0

Replication Case

This bug happens with each one of my planes and is not deticated to a single design.

KSP.log

KSP_x64.gc.log
KSP.log

@felixscheffer
Copy link
Contributor

I am working on a fix.

@morphelianus
Copy link
Author

After some test for the other bug I submitted I was able to reproduce the error more precisely. Whenever I engage the autopilot above 1000 meters everything is fine but below it sets the target altitude for the first nav point somewhere deep below base ground. I hope this helps a bit more. In case you need more details just leave a comment.

@felixscheffer
Copy link
Contributor

Thanks for the update.

I created a PR to fix this issue, so once the PR is merged it should be working again (in the dev build for now). I was using the wrong KSP method to convert the altitude back to a vector. This should only affect cases where the aircraft is below the glideslope when you engage autoland . If you start your descent above the glideslope, everything should be fine.

@morphelianus
Copy link
Author

morphelianus commented Jul 26, 2020

MechJeb Version (DEV RELEASE)

2.10.0.0-1001

Engaging aircraft approach and autoland below the slope sets the target altitude now correctly.

However, I recognized a short drop of the current vertical speed whenever I engaged aircraft approach and autoland after I engaged aircraft autopilot. Sometimes it could even drop into negative values. After a couple of seconds the current vertical speed matches again the target vertical speed and everything is fine.

@felixscheffer
Shall I open a new issue?

@felixscheffer
Copy link
Contributor

@felixscheffer
Shall I open a new issue?

Yes please

@morphelianus
Copy link
Author

@felixscheffer
Shall I open a new issue?

Yes please

Done. Nevertheless, this issue is solved. Well done.

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

Successfully merging a pull request may close this issue.

2 participants