Problems with adding UBs.

Wintermoore · Dec 2, 2014

Not sure if this is the right place to ask, but I haven't had any luck with anything anywhere else.

I built a custom Civ through Kael's guide, and it works alright (i.e. I can play it and it has unique units etc.), but I cannot add any unique buildings; when I start it up, the buildings are the default ones, and if I put the BuildingClassOverides bit anywhere else, it simply removes the buildings from the game.

Buildings.xml (note that I have not changed any info of the buildings but their type and cost.)

Spoiler :

LeeS · Dec 3, 2014

No such column as <SCIENCEKept>. I'm pretty sure this will cause the game to discard the file. The correct command for an Aqueduct-type building is <FoodKept>. You cannot change any of the Firaxis-supplied column names such as <Type>, <Cost>, <FoodKept>. You have to use what Firaxis used and you have to show them exactly as Firaxis used them. I.E., Capitalization is important, so that <FoodKept> is not at all the same thing as <foodkept> or <Foodkept>.

---------------------------------------------------------------------------------------------------------------------------------------------

The following is no good, but will not cause the game to discard the file. Coast and Ocean cannot be specified in that table <Building_TerrainYieldChanges>

Code:

<Row>
	<BuildingType>BUILDING_AETHER_MANIPULATOR</BuildingType>
	<TerrainType>TERRAIN_COAST</TerrainType>
	<YieldType>YIELD_SCIENCE</YieldType>
	<Yield>5</Yield>
</Row>
<Row>
	<BuildingType>BUILDING_AETHER_MANIPULATOR</BuildingType>
	<TerrainType>TERRAIN_COAST</TerrainType>
	<YieldType>YIELD_PRODUCTION</YieldType>
	<Yield>5</Yield>
</Row>
<Row>
	<BuildingType>BUILDING_AETHER_MANIPULATOR</BuildingType>
	<TerrainType>TERRAIN_OCEAN</TerrainType>
	<YieldType>YIELD_SCIENCE</YieldType>
	<Yield>5</Yield>
</Row>
<Row>
	<BuildingType>BUILDING_AETHER_MANIPULATOR</BuildingType>
	<TerrainType>TERRAIN_OCEAN</TerrainType>
	<YieldType>YIELD_PRODUCTION</YieldType>
	<Yield>5</Yield>
</Row>

There is a specific table for "Sea" tiles, called <Building_SeaPlotYieldChanges>. You cannot differentiate between Ocean tiles and Coastal tiles.

----------------------------------------------------------------------------------------------------------------------------------------------------

This is also no good, but will not cause the game to discard the file:

Code:

<Building_YieldModifiers>
	<Row>
		<BuildingType>BUILDING_TELARC_RESEARCH_COMPLEX</BuildingType>
		<YieldType>YIELD_CULTURE</YieldType>
		<Yield>100</Yield>
	</Row>
	<Row>
		<BuildingType>BUILDING_TELARC_RESEARCH_COMPLEX</BuildingType>
		<YieldType>YIELD_FAITH</YieldType>
		<Yield>60</Yield>
	</Row>
</Building_YieldModifiers>

There is a specific command within the <Buildings> table to accomplish the culture effect. The command would be

Code:

<CultureRateModifier>100</CultureRateModifier>

There is no command to accomplish the effect for Faith, and Faith does not work in the <Building_YieldModifiers> table.

Wintermoore · Dec 4, 2014

LeeS said:
snip

Ah, yes. A result of my shoddy Ctrl-F replacement, no doubt. Thank you.

The buildings do work now, but I am encountering a new problem: if I select my Civ in the setup screen, the UI disappears and I am left with an empty Civ background. I can still select it in the Advanced Setup.

ThorHammerz · Dec 4, 2014

LeeS said:
There is no command to accomplish the effect for Faith, and Faith does not work in the <Building_YieldModifiers> table.

Looks like it was intentionally excluded for whatever reason. Think it might be fairly simple to override/work-around however; in CvPlayer.cpp at line ~21249:

Spoiler :

Seems like the faith thing is just an explicit case: I assume just including a few lines of the default case

Spoiler :

into the case: FAITH would solve the issue?

LeeS · Dec 4, 2014

ThorHammerz said:

Looks like it was intentionally excluded for whatever reason. Think it might be fairly simple to override/work-around however; in CvPlayer.cpp at line ~21249:

Spoiler :

Seems like the faith thing is just an explicit case: I assume just including a few lines of the default case

Spoiler :

into the case: FAITH would solve the issue?

DLL (C++ ??) is just way over my head. I know there are numerous cases where Yield_X was not implimented to Table_Y. Whoward and others have been working on a community unified DLL patch to fix all those dangling and missing yield implimentations, but I'm not sure what the status is at the moment. Here is the thread for the DLL - Unified Yields.

ThorHammerz · Dec 4, 2014

LeeS said:
DLL (C++ ??) is just way over my head. I know there are numerous cases where Yield_X was not implimented to Table_Y. Whoward and others have been working on a community unified DLL patch to fix all those dangling and missing yield implimentations, but I'm not sure what the status is at the moment. Here is the thread for the DLL - Unified Yields.

A quick translation into English (or as close as I can get it :lol:

) :

Spoiler :

whoward69 · Dec 4, 2014

LeeS said:
but I'm not sure what the status is at the moment.

It's completed. If using mine or the CP DLL, every standard yield (food, gold, science, production, culture and faith) behave the same in ALL tables.

Also, two new yields - Tourism and GA Points - also behave the same in ALL tables

Additionally, there are 56 new secondary yield tables to "fill in the holes", eg if there was only a Belief_ResourceYieldChanges table, the tables Policy_ResourceYieldChanges and Trait_ResourceYieldChanges have been added - there are still a few gaps (see the table in post #2 of the linked thread) but they're just not worth the effort to implement

Problems with adding UBs.

Wintermoore

Chieftain

LeeS

Imperator

Wintermoore

Chieftain

ThorHammerz

zzz

LeeS

Imperator

ThorHammerz

zzz

whoward69

DLL Minion

Similar threads