MantisBT - VCMI
View Issue Details
0002417VCMIMechanics - Battlespublic2016-02-22 01:432024-01-30 19:18
SXX 
 
normalfeatureN/A
closedopen 
0.98f 
 
0002417: Feature: advanced garrison and visiting hero army merging for siege
So this one is ideas about army merging separated from 0000169. Currently I implemented H3 army merging in PR196 and it's will be used by default since it's what everyone used to.

Still H3 merging have big downside for defender since it's not automatically merge heroes stacks of same type when there is not enough space. It's also only check AIValue and don't take things like morale into account.

So idea is to implement multiple merging options so everyone can choose what they like. Some of them may be:

* No merging, e.g don't put visiting hero in garrison at all or only use creatures that belong to hero.
* H3 merging.
* H3 mechanics plus same type stacks merging if there not enough slots.
* Manual merging so before battle started defender get garrison window where he can choose creatures that going to participate battle.
No tags attached.
related to 0000169closed SXX The visiting hero does not collect the best troops from garrison to fill in his empty army slots when the town is under siege. 
related to 0002344resolved Ivan Dwellings: auto stack merge needed like we have with joining creatures on adventure map 
Issue History
2016-02-22 01:43SXXNew Issue
2016-02-22 01:43SXXRelationship addedrelated to 0000169
2016-10-16 11:00SXXRelationship addedrelated to 0002344
2023-04-12 17:06IvanStatusnew => confirmed
2023-04-12 17:06IvanNote Added: 0008570
2024-01-30 19:18IvanNote Added: 0008911
2024-01-30 19:18IvanStatusconfirmed => closed

Notes
(0008570)
Ivan   
2023-04-12 17:06   
Migrated to github, status will be tracked here:
https://github.com/vcmi/vcmi/issues/1219 [^]
(0008911)
Ivan   
2024-01-30 19:18   
Closing issues that were migrated to Github