Opened 3 years ago

Last modified 3 years ago

#6250 closed defect

PetraAI produces too much siege and ships. — at Version 7

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 Freagarach)

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. (​Phab:D4193)

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.

Change History (7)

comment:1 by Stan, 3 years ago

Milestone: Backlog → Alpha 25
Priority: Should Have → Release Blocker

comment:2 by Silier, 3 years ago

Component: Core engine → AI

comment:3 by Silier, 3 years ago

Owner: set to Freagarach

comment:4 by Freagarach, 3 years ago

In 25826:

(Partly) Fix PetraAI building a lot of siege and not using it.

Following r25584 / rP25584.
The used classes for adding siege were wrong.

Reported in ​https://wildfiregames.com/forum/topic/45178-a25-svn-version-25810-not-playable/.
Differential revision: ​https://code.wildfiregames.com/D4184
Refs. #6250 by (partly) fixing the siege case.
Reviewed by: @Angen

comment:5 by Freagarach, 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 Silier, 3 years ago

Cc: Silier added

comment:7 by Freagarach, 3 years ago

Description: modified (diff)
Patch: Phab:D4184 → Phab:D4184; Phab:D4193
Note: See TracTickets for help on using tickets.