Opened 3 years ago
Closed 3 years ago
#6250 closed defect (fixed)
PetraAI produces too much siege and ships.
Reported by: | Stan | Owned by: | Freagarach |
---|---|---|---|
Priority: | Release Blocker | Milestone: | Alpha 25 |
Component: | AI | Keywords: | regression |
Cc: | Freagarach, Silier | Patch: | Phab:D4184; Phab:D4193 |
Description (last modified by )
See https://wildfiregames.com/forum/topic/45178-a25-svn-version-25810-not-playable/. Probably introduced in r25584.
- She constructs a lot of siege without using it. (Uses most of it after r25826.)
- She constructs way too much ships without using them. (Fixed in r25828.)
For the first point, it seems that a lot of attack plans are present, since AttackPlan.addSiegeUnits is called quite a lot, even though Siege is constructed.
The second point was the result of ships having class Ranged and thus PetraAI training them when asked to train Ranged units. Fixed by trying to train Infantry+Ranged
.
Change History (10)
comment:1 by , 3 years ago
Milestone: | Backlog → Alpha 25 |
---|---|
Priority: | Should Have → Release Blocker |
comment:2 by , 3 years ago
Component: | Core engine → AI |
---|
comment:3 by , 3 years ago
Owner: | set to |
---|
comment:4 by , 3 years ago
comment:5 by , 3 years ago
Keywords: | regression added |
---|---|
Milestone: | Alpha 25 → Alpha 26 |
Pushing the rest of te ticket back to the next cycle.
comment:6 by , 3 years ago
Cc: | added |
---|
comment:7 by , 3 years ago
Description: | modified (diff) |
---|---|
Patch: | Phab:D4184 → Phab:D4184; Phab:D4193 |
comment:9 by , 3 years ago
Description: | modified (diff) |
---|---|
Priority: | Release Blocker → Should Have |
Decreasing priority since PetraAI works as intended again. She just makes plenty of siege.
comment:10 by , 3 years ago
Milestone: | Alpha 26 → Alpha 25 |
---|---|
Priority: | Should Have → Release Blocker |
Resolution: | → fixed |
Status: | new → closed |
Actually fixed in the two mentioned commits. The plenty of siege is a feature. ;)
If it is found that she indeed produces too much siege equipment, we can create a ticket for that since it existed probably in earlier alphas already. I'm closing this as fixed since the points mentioned and fixed were caused by r25584.
In 25826: