search

artwhaley / krpcmj

8 stars 3 forks source link

krpc.error.RPCError: Method not found: 'PartModuleList.GetEnumerator'. #3

Open jonasrosland opened 7 years ago

jonasrosland commented 7 years ago

When I try to use your simple launch example from the wiki with KSP 1.2.2 and kRPC 0.3.7, I get the following error message:

$ python3 simplelaunch.py
Traceback (most recent call last):
  File "simplelaunch.py", line 8, in <module>
    mj.ascent_alt=85000                          ##Orbital Altitude:
  File "<string>", line 1, in <lambda>
  File "/usr/local/lib/python3.5/site-packages/krpc/client.py", line 88, in _invoke
    raise RPCError(response.error)
krpc.error.RPCError: Method not found: 'PartModuleList.GetEnumerator'.

Any idea what's wrong?

My ckan list:

KSP Version: 1.2.2.1622

Installed Modules:

- KerbalEngineerRedux 1.1.2.8
- kRPC v0.3.7
- krpcmj autodetected dll
- KSPSteamCtrlr autodetected dll
- MechJeb2 2.6.0.0
- MechJebForAll 1.2.0.0
- ModuleManager 2.7.5
- Steamworks autodetected dll
djungelorm commented 7 years ago

Looks like krpcmj needs updating for KSP 1.2.2

jonasrosland commented 7 years ago

That would be great!

mathuin commented 6 years ago

Out of curiosity, where would one find out what one would need to know to update this project?

https://wiki.kerbalspaceprogram.com/wiki/1.2.2 and friends seem a little light on the details. https://kerbalspaceprogram.com/api/class_part_module_list.html shows the existence of an inline GetEnumerator() method in the PartModuleList class. Is this last site out of date?

djungelorm commented 6 years ago

It appears to work with KSP 1.3.1, if you use the changes in my pull request (https://github.com/artwhaley/krpcmj/pull/4)

Here's a binary so you don't have to compile it yourself: https://www.dropbox.com/s/ob6hji3e8awdxab/krpcmj.dll?dl=0

mathuin commented 6 years ago

The one tweak I had to make to your pull request was with regard to Partials/Rover.cs as it was missing. Other than that, I was able to build the DLL and it "worked" -- crashed my lander on the Mun but all the MJ magic got it there at least!

mathuin commented 6 years ago

@artwhaley noticed that #4 was merged, which should fix this issue. Can this issue be closed as a result?