Hotfix for 2-13 version

Status
Not open for further replies.

Gazebo

Lord of the Community Patch
Supporter
Joined
Sep 26, 2010
Messages
18,400
Location
Little Rock
Hey all,

Quick hotfix for 2-13 version. Savegame compatible.

  • Fixed small quirk that was preventing AI from improving luxuries.
  • Fixed DOW issue where unit capturing wasn't respecting capture classes properly

Uploaded as of 1pm EST. CP is only file affected (but also updated autoinstaller for those that want it).

Downloads Folder


Cheers,
Gazebo
 
You forgot to make installer remove the LUA folder from Community Patch (1) folder, which made the city icon not appear if one just installs it regularly.

All is fine upon deletion of the LUA folder
 
One thing I forgot to ask about, because I was testing other stuff, that has been happening since at least the last beta version already is that since then, using Really Advanced Setup is somewhat impossible if you want not play with all random civs.

What happens is that, as soon as I select any civ for anyone during setup, it fails to load the second unique icon for that civ and breaks selecting other civs or replacing the selected one until deletion of the RAS settings from the ModUserData folder. Which kiiind of defeats the purpose of why I like to use it. The issue is accompanied by this error in the lua.log:
Code:
[362882.271] Runtime Error: C:\Users\Mir Teiwaz\Documents\My Games\Sid Meier's Civilization 5\MODS\Really Advanced Setup (v 15)\UI\Main\GTAS_CivPanel.lua:204: bad argument #2 to '?' (Key must be of type 'number' or 'string'.)
Said section of code is as such, in case that gives any clues to what caused it in the first place:
Code:
for override in GameInfo.Civilization_UnitClassOverrides("CivilizationType = '" .. civ.Type .. "'") do
	if count >= MAX_CIV_BUTTONS then
		break;
	end
	local unit = GameInfo.Units[override.UnitType]; <-- This is the line causing problems, maybe related to the siege tower or some other unit change? Skirmisher?
	if unit then
		count = count + 1;
		local instance = slot.civManager:GetInstance();
		IconHookup(unit.PortraitIndex, 64, unit.IconAtlas, instance.CivPortrait);
		SetIconHelp(instance.CivPortrait, unit.Description, unit.Help);
	end
end

It might be a better idea to bring that up to the RAS dev, but since it only started happening after the recent betas, I figured I'd drop it here first. This happens with CBP & EUI + RAS, no other mods and at least since the Feb. 10th version, that's when I first noticed it.
 
One thing I forgot to ask about, because I was testing other stuff, that has been happening since at least the last beta version already is that since then, using Really Advanced Setup is somewhat impossible if you want not play with all random civs.

What happens is that, as soon as I select any civ for anyone during setup, it fails to load the second unique icon for that civ and breaks selecting other civs or replacing the selected one until deletion of the RAS settings from the ModUserData folder. Which kiiind of defeats the purpose of why I like to use it. The issue is accompanied by this error in the lua.log:
Code:
[362882.271] Runtime Error: C:\Users\Mir Teiwaz\Documents\My Games\Sid Meier's Civilization 5\MODS\Really Advanced Setup (v 15)\UI\Main\GTAS_CivPanel.lua:204: bad argument #2 to '?' (Key must be of type 'number' or 'string'.)
Said section of code is as such, in case that gives any clues to what caused it in the first place:
Code:
for override in GameInfo.Civilization_UnitClassOverrides("CivilizationType = '" .. civ.Type .. "'") do
	if count >= MAX_CIV_BUTTONS then
		break;
	end
	local unit = GameInfo.Units[override.UnitType]; <-- This is the line causing problems, maybe related to the siege tower or some other unit change? Skirmisher?
	if unit then
		count = count + 1;
		local instance = slot.civManager:GetInstance();
		IconHookup(unit.PortraitIndex, 64, unit.IconAtlas, instance.CivPortrait);
		SetIconHelp(instance.CivPortrait, unit.Description, unit.Help);
	end
end

It might be a better idea to bring that up to the RAS dev, but since it only started happening after the recent betas, I figured I'd drop it here first. This happens with CBP & EUI + RAS, no other mods and at least since the Feb. 10th version, that's when I first noticed it.

Still happening in latest?
 
Some heavy yield focus bug. The city prefers 3f tile over 4f1p tile, prefers 2f1p tiles over 2f3p tiles, 3f1p over 4f1p etc. It's as if it considered production a detriment to the city and was avoiding it like a plague?


Look at Barcelona, play around with the focuses. Same happens to other cities, you can't trust the AI now to not choose the inferior variants.EVEN on production focus the city would rather work 3f tile over 2f3p one now or 2f1p

View attachment barcelonaIsabella_0104 BC-1400.Civ5Save
 
Some heavy yield focus bug. The city prefers 3f tile over 4f1p tile, prefers 2f1p tiles over 2f3p tiles, 3f1p over 4f1p etc. It's as if it considered production a detriment to the city and was avoiding it like a plague?


Look at Barcelona, play around with the focuses. Same happens to other cities, you can't trust the AI now to not choose the inferior variants.EVEN on production focus the city would rather work 3f tile over 2f3p one now or 2f1p

View attachment 404081

Good catch - I see the bug. Plot valuation was inflating if you had a pantheon in certain situations. A simple bug, but obscure.

Fixed and uploading now (savegame compatible). Just the CP.

Use link on main page.
 
I play with 12 AI at Prince difficulty. Even after 200 turns, no one denounce anyone. There are wars everywhere,but no denounce?
 
Bug (posting here cuz it is most likely this beta which added it) - Holy Site no longer gets +1 Food from Tradition finisher.

I remember it used to about a beta ago but now it doesn't.
 
Bug: turn 109 the world congress has not been built yet suddenly a bunch of civs messaged me about not liking my congress proposal (-15)
 
Does the Statecraft finisher still allow you to purchase Great Diplomats at the Industrial Era? Or was this ever reverted? Anyone? I need confirmation since I can't purchase them.
 
I can buy other Great Person types, no problem. Great Diplomats simply don't appear on the list of purchasable units. There's a majority religion of course, and I'm using CSD of course. I am not using any other mods apart from the 6 mods for the full CP experience. There might be something wrong with my installation but I can't see what I did wrong. I already reinstalled it several times. I took the usual steps I always did and there were no problems until now. And it's a very minor issue that's easy to miss.

A screenshot of anyone being able to purchase Great Diplomats in this version should be enough proof that I did something wrong though.
 
Dearth of feedback around here.

How are the new tech tree and warmonger tooltips?

G

Tech tree tooltips are nice, although I never had problems with the limited buttons (or maybe I missed the problems).

Warmonger tooltip is also nice, but I am not sure they justify the loss of very valuable, graphic city info banner tooltips on CP-only setups... why is it that CP-only cannot have the same tooltip as CBP full?
 
Dearth of feedback around here.

How are the new tech tree and warmonger tooltips?

G

They rock. :cool:

Seriously, the new tech tree tooltips is awesome because it lists what the tech does, without having to mouse over a specific spot on the tech and pray the UI knows what you want. You just mouse over the tech and it lists all of it.

The warmonger tooltips are also awesome! They tell you how each civ will react to the cities you take, and they correctly show that your target and your target's friends will penalize you more than neutral civs, who in turn penalize you more than your friends and war allies (they barely notice :cool:).

The only flaw seems to be I can't find how to view the tool tips before capturing the city - I just get the usual Minor/Major/Extreme warning. I can only see the new tool tip when I'm selecting Annex/Puppet/Raze.
 
They rock. :cool:

Seriously, the new tech tree tooltips is awesome because it lists what the tech does, without having to mouse over a specific spot on the tech and pray the UI knows what you want. You just mouse over the tech and it lists all of it.

The warmonger tooltips are also awesome! They tell you how each civ will react to the cities you take, and they correctly show that your target and your target's friends will penalize you more than neutral civs, who in turn penalize you more than your friends and war allies (they barely notice :cool:).

The only flaw seems to be I can't find how to view the tool tips before capturing the city - I just get the usual Minor/Major/Extreme warning. I can only see the new tool tip when I'm selecting Annex/Puppet/Raze.

Should be there on hover (hover over the citybanner while at war). If that's not working, sounds like one of your mods are overwriting the CP's banner.

Warmonger tooltip is also nice, but I am not sure they justify the loss of very valuable, graphic city info banner tooltips on CP-only setups... why is it that CP-only cannot have the same tooltip as CBP full?

Not sure what you are talking about...if you use the citybanner in the CP lua folder, you get all the info. If you delete it, you get the old warmonger tooltip.
 
Status
Not open for further replies.
Back
Top Bottom