-
Notifications
You must be signed in to change notification settings - Fork 33
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
Refactor the legions code #16 #85
Conversation
- Use always = no for legions hire trigger in landed_titles, so that they are not visible from other rulers. Vassalized mercenaries are always hirable by their liege, whatever the allow trigger. - Remove condition on e_wre, e_ere or k_soisson: any Romance culture character with matching historical Roman title can re-create the legion. - Removed usage of global flags, as they werer missing in title history, and not really needed anyway. - Move completely_controls from potential to allow trigger, to give clue to the player if some parts are not controlled - Workaround count tier mercenary companies that cannot be destroyed, by making the legion independent (without possibility to be hired by anyone). - Reduced legion disband condition from completely_controls title, to has title and has Romance culture, so that legion won't be destroyed by temporary revolts, or minor land loss. - Extract duplicated Romance culture condition to scripped_triggers - Add 'organize legions' as a feature of Romance cultures - Add missing decisions / events for Eastern legions - Add localization for creation and disband legion narrative events. - Merge WtWSMS_western_legions_on_restoration.txt into WtWSMS_legion_events.txt - Split decisions/events for multiple legions at a time (Dalmatia and Africa) into multiple decisions/events, for consistancy and because cost was unaffordable.
Looks good! Changed the title in order to make it reference to the dedicated issue. I will let @EWattman look into this, as he was the one who reimplemented them and discussed the conditions. |
The Armenians should be able to use them as well, if they are properly Roman Christian or Hellenic Pagan (+Heresies), in essence within the Roman Sphere of Influence and not the Persian. I want the Legions to be restorable by King-level Romans or above. Meaning Emperors and Pretender Kings, so that Soissons can restore them, but currently no one else. Also, could some event-thing be added so that the newly-restored legions fall under the control of however owns, for example, the Duchy of Mesopotamia and not automatically the liege? |
|
2/3. This is my way of simulating the Legions being stationed throughout the Empire, rather than having them all at Constantinople, making the Emperor invulnerable. Ideally, the Legions would be vassalized under however controls their home territory, which in-game translates into whoever controls the Duchy/Kingdom-title they have as creation-requirement. |
|
That is too bad, because the current Dejure Kingdoms that we have, the Dioceses, were more civil in administation than military. For example, the IV Parthica, in the 5th century, is under the command of the governor of Osroene in Edessa, not the Comes in Antioch. And Osroene is a duchy in the game. Look, if we can't find a solution to that problem, we will return the Legions to Duchy-level for the sake of game mechanics. |
I'll do some more testing, as apparently de_jure_liege only works with n+1 tier.
|
Have you tried the d_salona or d_andautonia? We could spread the non-historical Provincial Legions out all we want. |
- Set de_jure_liege via province history (note: hast to be a duchy tier title) - Workaround destroy/deactivate of count tier titles not fully working, by killing the owner.
Ok, it works when using duchy as de_jure_liege !
So I've set their duchies for some historical legions, using title history (you'll need to double check / balance). Not sure about the legions controlled by the emperor directly: could be either dejure to the capital duchy, or have no dejure. The requirement completely_controls = e_byzantium for some decisions might be a bit too strong. Also I've found a workaround for the destroy_title not working (need to kill the title owner). |
I fixed that conflict, and Erik said it was OK to merge so I will merge it and close down. |
I've refactored the legions code to solve some issues/discrepancies, could you please review ?
I've discovered that:
I've extracted the list of Romance cultures (that can re-organize legions and won't cause legions to disband). There was some condition on Armenian and some religions in the previous allow trigger, I'm not sure what to do with this.
Changes: