bszonye/civ6-capslock

Caps Lock mod for Civilization VI

civ6-capslock's Issues

New terrain check causes type error

hexal reports:

After the last update this is no longer compatible with Got Lakes Map Script. City states and tribal villages will not spawn.

I was able to reproduce, with this error:

Runtime Error: C:\Users\bradd\Documents\My Games\Sid Meier's Civilization VI\Mods\capslock\AssignStartingPlots.lua:567: operator - is not supported for number - boolean
stack traceback:
        C:\Users\bradd\Documents\My Games\Sid Meier's Civilization VI\Mods\capslock\AssignStartingPlots.lua:567: in function 'AssignStartingPlots:__SetStartMinor'
        C:\Program Files (x86)\Steam\steamapps\workshop\content\289070\916614621\GotLakes.lua:13424: in function 'AssignStartingPlots:__InitStartingData'
        C:\Users\bradd\Documents\My Games\Sid Meier's Civilization VI\Mods\capslock\AssignStartingPlots.lua:79: in function 'AssignStartingPlots.Create'
        C:\Program Files (x86)\Steam\steamapps\workshop\content\289070\916614621\GotLakes.lua:16647: in function 'DetermineStartingPlots'
        C:\Program Files (x86)\Steam\steamapps\workshop\content\289070\916614621\GotLakes.lua:16676: in function 'GenerateMap'
    [C]: in function '(anonymous)'

It looks like Got Lakes is overriding some of the functions in the AssignStartingPlots object, and failing because the interface is no longer consistent (number score instead of boolean check).

Fix weights for polar starts & adjacent terrain

The self:__GetValidAdjacent check is only given +1 weight. That function checks both for polar starts and for bad adjacent terrain. The low weight means that civs can still end up in polar snow, especially if the major civ buffer produces a high penalty score.

YnAMP compatibility

The YnAMP mod also overrides AssignStartingPlots.lua with LoadOrder=2. Therefore, Caps Lock will break their True Start Location scripting at least, and possibly more.

Some possible resolutions:

• Mark YnAMP as an incompatible mod to avoid broken combinations.
• Reduce the Caps Lock LoadOrder from 12,000 to 1 so that YnAMP takes precedence.
• Contribute the Caps Lock improvements to YnAMP and coordinate LoadOrder.

The first two resolutions would also enable a “Caps Lock + YnAMP” compatibility shim that combines the two mods, but I would prefer to avoid the parallel coding that would entail.