-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Review and act on assorted CE changes. #78
Comments
You may also be interested in Skeletonxf/star-ruler-2-mod-ce@a1d04dd These three might have only happened because of how I messed with config settings for the invasion map This one basically requires pulling in my entire generic FTL AI component (though would also allow the AI to use a teammates fling beacons without teaching the AI the prototype -> full unlock steps for FTL in rising stars) This one also I think is a fix for something that can't happen in vanilla. The mini worm holes are the only oddities without strategic icons, I may have effectively created this bug by making the icon nullable Cargo orders is quite a clean slice of just a few files (Context menu, LeaderAI, and the order files iirc), so you'll have an easier time porting it by looking at the relevant file's master branch versions than going through each diff. I recall us discussing what sort of logic for terraforming undos would be appropriate, you might have wanted a different algorithm for this one This just did't work and I'm not sure why This remains buggy and I might have to give up on getting outposts to work with the Things in Combat quickbar. |
I don't see why I would be. Unlike CE, very little in RS still depends on the combat timer, and I am reluctant to encourage artillery much further.
How so?
On the AI front, I'm actually waiting until Dragon's a little more complete, so I don't have to keep re-porting it too often. But for some reason I can't remember, I never implemented fling sharing, only gate sharing.
Got it.
Some of the files are cleaner than others, but I'd like to see how it evolved, either way.
Possibly, yes... I'll have to look at it more closely. |
That makes sense. I do think vision granting from damage dealing is a useful mechanic regardless of where it is bolted onto (in CE, the combat timer seemed the obvious place). I suppose RS already has some piecemeal aspects of this from things like boarding parties.
I've only ever hit those null pointers using a save file where the config I added to the invasion map let us spawn it in without any players or homeworlds and not even based on the number of players in the game. I suppose thinking about it, it might still happen in vanilla if lack of homeworld in the galaxy is the trigger so perhaps best to port the fixes anyway.
Because Fling sharing made Gate sharing look easy :P Makes sense on waiting on the dragon AI. However I have no idea what kind of problems if any porting the sharing without at least updating the AI's gate/fling checks of can I use this could have. I had to review pretty much every AI gate/fling line of code (not actually that many of them, but I didn't check the dead code in the single FTL files) to ensure it would still be getting the appropriate list of owned or usable gates/flings. Don't want the AI deciding it can move a gate it doesn't own after all. |
The text was updated successfully, but these errors were encountered: