Can't make a node in maneuver planner operations about krpc.mechjeb HOT 6 CLOSED

MechJeb 2.9.2.0 added support for Multi-Node maneuvers and deleted the MakeNode() method which my code wants to call.

Modifications need to be made to all operations; see MuMech/MechJeb2@44ccd89.

from krpc.mechjeb.

This modification causes an error for MechJeb < 2.9.2.0, because now the code tries to call MakeNodes() instead of MakeNode(). Just a suggestion but some backward compatibility would be great.

from krpc.mechjeb.

@EnJiang Thank you for your suggestion. Sometimes it's difficult to keep things backward-compatible, especially in "pre-release" versions (kRPC.MechJeb API is not yet finalized, though it's coming close to v1.0) and it would be too much work to keep all things backward compatible, if not impossible (see commit 95603ab).

When I fixed this issue, I thought that backward compatibility is important mainly for client code, so I kept MakeNode() and marked it as deprecated. Why do you think that I should consider backward compatibility for MechJeb versions? Is there something that prevents you from updating MechJeb to the newer version?

from krpc.mechjeb.

No. In my case I just downgraded kRPC.MechJeb. It's just it took me some time to google and figure out why I can't make a node and it's a little bit frustrating. If backward compatibility is not economical to do, maybe some clearer indications telling the user maneuver planner won't work perfectly for this version MechJeb would also be good.
All of these are pure optional : )
BTW, I've been wanting a programmatic way of MechJeb manipulating for a long time, your work is so great absolutely love it!

from krpc.mechjeb.

@EnJiang If I may ask, why did you rather downgrade kRPC.MechJeb than upgrade the MechJeb2 itself?

By the way, on the Releases page, there is a compatible kRPC and MechJeb2 version for recent releases. Do you think I should mention it somewhere else as well or write other details?

from krpc.mechjeb.

@Genhis I guess it was just a bit handy because I had kRPC.MechJeb repo opened. I think I'll upgrade all of them to the latest version now.

By the way, on the Releases page, there is a compatible kRPC and MechJeb2 version for recent releases. Do you think I should mention it somewhere else as well or write other details?

Thank you! I will follow that.

from krpc.mechjeb.