| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
|---|---|---|---|---|---|---|---|---|---|
| 0005530 | AI | AI | public | 2017-04-22 01:42 | 2017-04-25 18:06 | ||||
| Reporter | ThinkSome | ||||||||
| Assigned To | abma | ||||||||
| Priority | normal | Severity | crash | Reproducibility | always | ||||
| Status | resolved | Resolution | fixed | ||||||
| Summary | 0005530: AAI crashes spring 103.0 with Spring:1944 on expanded confluence | ||||||||
| Description | Can we now get ValidAIs.lua implemented? Even if these AIs didn't crash we would avoid people being puzzled over why the supposedly super-duper (based on description) AI they've picked does not do anything at all. https://springrts.com/mantis/view.php?id=2923 | ||||||||
| Tags | No tags attached. | ||||||||
| Attached Files |
| ||||||||
 Relationships |
|
 Relationships |
| Notes |
|
|
ThinkSome (reporter) 2017-04-22 01:43 |
demo, not much to look at. |
|
AF (reporter) 2017-04-22 12:36 |
https://github.com/spring/spring/blob/fc3551308c65e90c67f27340267cc2bb537ff605/AI/Skirmish/AAI/AAI.cpp#L184 AAI is deliberately crashing here to indicate the game is unsupported. While a new callback to indicate to the engine if the game is supported would be useful so it never showed up as an AI in the first place, AAI should make it clear via messages then disable itself. A boolean on the main AAI object, and a check to return early if it's set to false would allow this until a more effective solution is found |
|
Anarchid (reporter) 2017-04-25 12:45 |
It could also surrender instead of crashing the game (and lobby if it's an ingame one!). |
|
abma (administrator) 2017-04-25 13:44 |
(partly?) fixed: https://github.com/spring/spring/commit/488c9de3bfd7b3568af1ab6b09d96349f8b56a89 does it still crash when running AAI with S44? |
|
AF (reporter) 2017-04-25 14:19 |
That would fix the crash, but who knows what AAI will do afterwards? I like Anarchids suggestion of surrendering the game, it prevents the player building a big army and going across the map to find the starting units sitting idle. There's no surrender API call but giving every unit a self destruct command should do |
| Notes |
| Issue History |
|||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2017-04-22 01:42 | ThinkSome | New Issue | |
| 2017-04-22 01:42 | ThinkSome | File Added: infolog.txt | |
| 2017-04-22 01:43 | ThinkSome | File Added: 20170422_011706_Expanded_Confluence_103.sdfz | |
| 2017-04-22 01:43 | ThinkSome | Note Added: 0017472 | |
| 2017-04-22 12:36 | AF | Note Added: 0017475 | |
| 2017-04-25 12:45 | Anarchid | Note Added: 0017490 | |
| 2017-04-25 13:44 | abma | Status | new => feedback |
| 2017-04-25 13:44 | abma | Note Added: 0017492 | |
| 2017-04-25 14:19 | AF | Note Added: 0017493 | |
| 2017-04-25 18:06 | abma | Assigned To | => abma |
| 2017-04-25 18:06 | abma | Status | feedback => resolved |
| 2017-04-25 18:06 | abma | Resolution | open => fixed |
| Issue History |