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

1.4.x support #147

Open
ianglennon opened this issue Mar 11, 2018 · 13 comments
Open

1.4.x support #147

ianglennon opened this issue Mar 11, 2018 · 13 comments

Comments

@ianglennon
Copy link

Hi,

Great mod, really missing it now that KSP has upgraded to 1.4. Any news on an updated release?

@Jacalz
Copy link

Jacalz commented Mar 17, 2018

I would really want this too and also better support for making history expansion! 🙂

@cculianu
Copy link

Please please please add 1.4.1 support when you have time! This is my favorite mod!

@lnxbil
Copy link

lnxbil commented Mar 19, 2018

Besides the warning on game entry, it seems to work just fine. Have you experienced any problems?

@cculianu
Copy link

Oh hey... you're right. It works perfectly. Well.. color me surprised.

Awesome!

@jrbudda
Copy link
Contributor

jrbudda commented Mar 21, 2018

https://github.com/jrbudda/KerbalEngineer/releases/tag/1.1.4.0

@Jacalz
Copy link

Jacalz commented Mar 21, 2018

Aha, it looks like you have taken over the development in your own fork? @jrbudda

@jrbudda
Copy link
Contributor

jrbudda commented Mar 21, 2018

I don't really want to 'take over' but thought I'd spend a couple hours getting something out there working for folks. I did submit a PR for the changes, but it looks like PRs havent been pulled here since October so I uploaded a release on my fork.

@Jacalz
Copy link

Jacalz commented Mar 21, 2018

That is great and I wouldn’t say that taking over would be a bad thing since this repo is already pretty much dead but yeah 🙂 Great work though 👍

@cculianu
Copy link

@jrbudda DUDE!! I love you! Thank you for this!! YOU ARE THE BEST!!

If he doesn't respond in some time -- think about maybe submitting this to CKAN so that others may benefit?

Thanks again!!

@oneam
Copy link

oneam commented Mar 22, 2018

Thanks @jrbudda
Unfortunately, I found an issue with your update, if I attach RCS thrusters to an engine plate it throws:

[Exception 20:52:54.0799760]: SimManager.StartSimulation() // Object reference not set to an instance of an object
[]: at KerbalEngineer.VesselSimulator.PartSim.DecoupledInStage (.Part thePart) [0x00000] in :0
at KerbalEngineer.VesselSimulator.PartSim.New (.Part p, Int32 id, Double atmosphere, KerbalEngineer.LogMsg log) [0x00000] in :0
at KerbalEngineer.VesselSimulator.Simulation.PrepareSimulation (KerbalEngineer.LogMsg _log, System.Collections.Generic.List`1 parts, Double theGravity, Double theAtmosphere, Double theMach, Boolean dumpTree, Boolean vectoredThrust, Boolean fullThrust) [0x00000] in :0
at KerbalEngineer.VesselSimulator.SimManager.StartSimulation () [0x00000] in :0

@jrbudda
Copy link
Contributor

jrbudda commented Mar 22, 2018

Ah yea looks like anything surface-attach

@jrbudda
Copy link
Contributor

jrbudda commented Mar 22, 2018

should be fixed here https://github.com/jrbudda/KerbalEngineer/releases

You'd think FindAttachNodeByPart would never return null for a part that is actually attached and AttachNode's can be of type SURFACE. But you'd be wrong.

@cculianu
Copy link

Dude, you are awesome. Thanks for the quick turn-around on this.

As a fellow programmer, I can relate. Obvious API calls not doing the obvious. ;)

Ha ha -- good work though man. Thanks!!

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

6 participants