View Issue Details
| ID | Project | Category | Date Submitted | Last Update | |
|---|---|---|---|---|---|
| 0025624 | AI War 2 | Bug - Gameplay | Oct 7, 2021 12:38 am | Oct 13, 2021 6:07 pm | |
| Reporter | Eluthena | Assigned To | Chris_McElligottPark | ||
| Status | resolved | Resolution | fixed | ||
| Product Version | Beta 3.705 Blazing Collections And The New Spire | ||||
| Fixed in Version | Beta 3.708 Bugfixes And Death Spawn | ||||
| Summary | 0025624: Map generation is not applying map-type-specific customization options when they are changed in the lobby UI | ||||
| Description | In the custom start lobby, when I try to change map customization options such as map linking flavor/algorithm or planet layout, the changes do not seem to apply. Even after manually hitting regenerate map, it does not update, and pressing "start game" and looking at the galaxy map indicates that the preview is correct. Changing the number of planets, map type, planet naming scheme, and map seed all apply properly, but the map-type-specific customization options are still not applied in the regenerated map. Example: 1) Open custom start lobby, and press "Reset to Defaults" --> Map preview should update to reflect the default settings 2) Change "Customize: Planet Layout" to "Big Circle" and press "Regenerate Map" --> Preview does not update 3) Change "Desired Planets" to "120" and press "Regenerate Map" --> Preview updates to show the change in planet count, but is still not in the big circle layout Also, I'm not sure if this is related, but automatic map regeneration does not always trigger when changing specifically either Planet Naming Scheme or Desired Planets. For me it seems to skip every other change (e.g. change planet count to 90 and it updates, then change it to 100 and it does not update, then change it to 110 and it does update, and so on). System Information: CPU: Intel Core i7-8700 GPU: GTX 1060-3GB RAM: 16GB OS: Windows 10 (Version 21H1 / Build 19043.1237) | ||||
| Tags | No tags attached. | ||||
|
|
I was looking at the map generation code, and it appears that many `map_option`s don't have `related_int_value` set, despite the code trying to consume them that way (and thus always getting 0). |
|
|
I have also observed this happening |
|
|
I also have observed this. |
|
|
Thank you! * Fixed an issue with my list collections from two builds ago where FindAll and GetRange could throw an exception if they were to return a zero-length list. ** This was causing an exception in the Swirl map type. * From June 18th of this year, I made it so that map options were a lot more robust, but at the same time introduce a possibility where maps would not have an underlying related_int set properly. ** Because of how this was done, this made it really easy for modders to shoot themselves in the foot, and I also shot myself in the foot with a ton of the existing maps as I tried to convert the data over. I essentially broke the sub-options on a lot of map types at that point. ** To make sure that these don't just slide by unnoticed in the future, if the map generation algorithm tries to read an uninitialized integer or fixed-integer, it will now throw an exception. ** Finishing the fix in the short term, then went through and fixed up the xml for tons of map types from the base game and dlcs, which have been quietly not having their sub-options working for all this time, it turns out. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| Oct 7, 2021 12:39 am | Eluthena | New Issue | |
| Oct 7, 2021 10:25 pm | tom.prince | Note Added: 0062912 | |
| Oct 12, 2021 12:53 pm | BadgerBadger | Note Added: 0062919 | |
| Oct 13, 2021 5:34 pm | Chris_McElligottPark | Note Added: 0062926 | |
| Oct 13, 2021 6:07 pm | Chris_McElligottPark | Assigned To | => Chris_McElligottPark |
| Oct 13, 2021 6:07 pm | Chris_McElligottPark | Status | new => resolved |
| Oct 13, 2021 6:07 pm | Chris_McElligottPark | Resolution | open => fixed |
| Oct 13, 2021 6:07 pm | Chris_McElligottPark | Fixed in Version | => Beta 3.708 Bugfixes And Death Spawn |
| Oct 13, 2021 6:07 pm | Chris_McElligottPark | Note Added: 0062927 |
