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
Replaced civillians with Technicians when building sold #12174
Conversation
^ScienceBuilding should also emit "technicians" instead of "civilians" ... and we could make this a bit more interesting by having ^CivBuilding emit "civs" ... when they are destroyed ... with "EjectOnDeath:" so we can actually still see "Civilians" ingame |
@@ -466,7 +466,7 @@ | |||
ExternalCapturable: | |||
ExternalCapturableBar: | |||
EmitInfantryOnSell: | |||
ActorTypes: e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,c1,c2,c3,c4,c5,c6,c7,c8,c9,c10,e6,e6,e6,e6,e6 | |||
ActorTypes: e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,e1,tecn,tecn,tecn,tecn,tecn,tecn,tecn,tecn,tecn,tecn,e6,e6,e6,e6,e6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This list now contains 15x e1, 10x tecn and 5x e6, so you can actually shorten it drastically to e1,e1,e1,tecn,tecn,e6
to get the same probability ratio.
Although on that note, only construction yards should emit e6
, so maybe we should remove it here and only add it on FACT.
Changes made. I think that the ^CivBuildings emitting civs on destruction should be raised as a separate issue as changes need to be made to the EjectOnDeath trait. |
👍 |
👍 / ✅ |
Fixes #12122.