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

Ascent guidance doesn't like having Mammoth as first stage #1029

Closed
rsparkyc opened this issue May 15, 2018 · 4 comments
Closed

Ascent guidance doesn't like having Mammoth as first stage #1029

rsparkyc opened this issue May 15, 2018 · 4 comments

Comments

@rsparkyc
Copy link

rsparkyc commented May 15, 2018

Testing this with RSS installed (working on building a RP-0 installation) on 1.4.3 with build 800. Here's the exception:

MechJeb module MechJebModuleAscentAutopilot threw an exception in Drive: System.ArgumentOutOfRangeException: Argument is out of range.
Parameter name: index
  at System.Collections.Generic.List`1[MuMech.MechJebModuleAscentPEG+StageInfo].get_Item (Int32 index) [0x00000] in <filename unknown>:0
  at MuMech.MechJebModuleAscentPEG.UpdateRocketStats () [0x00000] in <filename unknown>:0
  at MuMech.MechJebModuleAscentPEG.DriveAscent (.FlightCtrlState s) [0x00000] in <filename unknown>:0
  at MuMech.MechJebModuleAscentAutopilot.DriveAscent (.FlightCtrlState s) [0x00000] in <filename unknown>:0
  at MuMech.MechJebModuleAscentAutopilot.Drive (.FlightCtrlState s) [0x00000] in <filename unknown>:0
  at MuMech.MechJebCore.Drive (.FlightCtrlState s) [0x00000] in <filename unknown>:0
@rsparkyc
Copy link
Author

I can confirm this is an issue when playing just stock + mechjeb

@rsparkyc
Copy link
Author

Actually, it looks like my number of stages may have been set to more than the rocket had, perhaps that's what was throwing the exception

@lamont-granquist
Copy link
Collaborator

https://github.com/lamont-granquist/MechJeb2/releases doesn't have num_stages (and generally the now-old PEG code in MJ release is going to get replaced rather than fixed)

@rsparkyc
Copy link
Author

Ahh, didn't realize you were doing mechjeb builds @lamont-granquist. Seeing how many of the most recent PRs have been from you, I suppose I can assume your builds will be more up to date than this this repo.

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

No branches or pull requests

2 participants