0003208: Ai hangs when it cannot recruit heroes over set limit

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0003208

VCMI

AI - Adventure Map

public

2022-04-14 13:27

2022-04-28 05:56

Reporter

val-gaav

Assigned To

Priority

normal

Severity

major

Reproducibility

always

Status

resolved

Resolution

fixed

Platform

Windows

OS Version

10 and 11

Product Version

Target Version

Fixed in Version

1.next

Summary

0003208: Ai hangs when it cannot recruit heroes over set limit

Description

There is a setting in config/defaultmods.json :
"MAX_HEROES_ON_MAP_PER_PLAYER" : 8,
If one changes it to anything lower then 8 then AI sooner or latter will hang up on this.

VCMIAI will just hang and loop trying to recruit hero end not end it turn.

Nullkiller AI will hang up on trying to recruit hero and it turn and on each next turn it will hang up on this over and over again.

Often when for example hero on map limit is set to 2 AI will end up buing 3 heroes and having them on map and then hanging up on this

Steps To Reproduce

Modify the file
config/defaultmods.json

and set to
"MAX_HEROES_ON_MAP_PER_PLAYER" : 2,

It will trigger this error always for both available adventure AIs

Additional Information

Nullkiler AI when I set:
"MAX_HEROES_ON_MAP_PER_PLAYER" : 10,

had no problems and ended up using 9 heroes on map.

Tags

No tags attached.

Attached Files

Relationships

Notes
(0008276) val-gaav (reporter) 2022-04-16 21:03	Just tested : "MAX_HEROES_AVAILABLE_PER_PLAYER" : 1, "MAX_HEROES_ON_MAP_PER_PLAYER" : 8, and "MAX_HEROES_AVAILABLE_PER_PLAYER" : 2, "MAX_HEROES_ON_MAP_PER_PLAYER" : 2, in hope that maybe "MAX_HEROES_AVAILABLE_PER_PLAYER" will be respected by AI, but the results are the same AI loops trying to hire more heroes.

(0008285) val-gaav (reporter) 2022-04-27 13:45	This is fixed

(0008286) Povelitel (updater) 2022-04-28 05:55	Fixed by val-gaav in https://github.com/vcmi/vcmi/pull/733 [^]

Issue History
Date Modified	Username	Field	Change
2022-04-14 13:27	val-gaav	New Issue
2022-04-16 21:03	val-gaav	Note Added: 0008276
2022-04-27 13:45	val-gaav	Note Added: 0008285
2022-04-28 05:55	Povelitel	Note Added: 0008286
2022-04-28 05:55	Povelitel	Status	new => resolved
2022-04-28 05:55	Povelitel	Fixed in Version	=> 1.next
2022-04-28 05:55	Povelitel	Resolution	open => fixed
2022-04-28 05:55	Povelitel	Assigned To	=> Povelitel
2022-04-28 05:56	Povelitel	Assigned To	Povelitel =>

Site | Forums | Wiki | Slack | GitHub