SirDiazo / AGExt

Action Groups Extended
13 stars 19 forks source link

Add RemoteTech Support #10

Open SirDiazo opened 10 years ago

SirDiazo commented 10 years ago

Add remotetech support

pjf commented 9 years ago

If RemoteTech support means disallowing action groups to be activated by the user when there's no connection, and allowing action groups to be scheduled in the flight computer, then I am so excited about this. Thank you. :)

I believe as a workaround to the second part one can use the features of AGExt where it writes to a stock action group, and then schedule that group in the flight computer.

SirDiazo commented 9 years ago

I have not looked at this and probably will not start work until RemoteTech is 0.90 compatible, but I was hoping to be able to actually hook into Remotech so all groups are available with all the same options as the base 10 groups.

Having not looked at RemoteTech's code, I have no clue how practical this is, but that is my intent.

pjf commented 9 years ago

I've not looked at the RT code either, although I discovered to my delight that AGExt lets me assign to groups such as "gears", and RemoteTech lets me schedule "gears" to activate at a given time, so my scheduled activation of all the equipment I needed via flight computer, after the vessel had already launched, worked a treat. :)

Thank you again for AGExt, it's continuing to be one of my most favourite mods.

SirDiazo commented 9 years ago

No problem, it's become my flagship mod at this point. Overtook my Vertical Velocity mod somewhere around 24.2 for that title.

Just a heads up that the gears trick works because AGExt handles the non-numbered action groups differently in that it lets KSP handle them. So you tell AGExt "Activate Group 1" it goes ahead and activates all actions in group 1 directly itself.

If you tell it "Activate Gears" (or lights/abort/etc.), AGExt tells KSP "Hey, activate the gears group" and does nothing itself.

SirDiazo commented 9 years ago

As of AGX 1.31 this should be complete.

Leaving it open until RemoteTech 1.7 official is released to make sure nothing changes on me before then.

SirDiazo commented 9 years ago

While this is working, should change the local check to use the new IsLocal method added to RT's API.