[New Feature] (beta) Cultural Components (5/6 UC)

[azum4roll]

I guess its not possible to use DELETE FROM... in DB.Query()? i've only used it once or twice to return values from the DB (ie via a select statement), but was curious what would happen if other SQL command was passed into it

That'll only create gaps in tables.

 

[jarcast2]

This makes it cost 1 pop in VP (cant separate the two effects atm)

You may just readd the citizen with lua.

 

[johnchen]

Sad news about the strategic view limitation, if that is the source.

It would be nice, but (as you know) the database is set before any civs are chosen.
Someone with real engineering chops would need to take a look -- I don't know if it is even possible within the DLL.


Hmm, that's new to me. Must be the assert was suppressed on 4.22. Could you raise a ticket on github with the mods uploaded and the savegame?


You did better with it than me, but I did Huns not Mongols, so yeah this Khan heal is a good combo.
Can certainly push the tech back. Military Strategy? Can drop the XP to, say, 75% at the same time.
What do you think?


Yes, it converts food to production when building.
This makes it cost 1 pop in VP (cant separate the two effects atm)
I would probably remove that part if I could. (hint hint)

Costing pop is making it a bit too weak. Let's hope it doesn't replace regular Rifleman?

 

[hokath]

It does. I had a good game with them, but I probably made only 1 or 2 -- the rest were upgrades.
That's probably not representative, so I'll disable it.
Would you take a look to see if you can split the column in the DLL? Or, check for UNITCOMBAT_SETTLER to decide whether to remove pop?
The custommodoption tag is BALANCE_SETTLERS_CONSUME_POPULATION

The lua solution is too janky, I think.

 

[Bysonte]

Hmm, that's new to me. Must be the assert was suppressed on 4.22. Could you raise a ticket on github with the mods uploaded and the savegame?

Do I post it on the main VP github or is there a specific 5/6 uc one?
A link will be greatly appreciated.

 

[hokath]

the main github

LoneGazebo/Community-Patch-DLL

Community Patch for Civilization V - Brave New World - LoneGazebo/Community-Patch-DLL

github.com

the form should prompt you to provide the relevant stuff

 

[msw1]

And on the 12th day of Christmas,
We fix the bugs


First, make sure you clear the cache. This needs to load afterwards because of Reindeer.
I'll do a quick update to make sure all the rows are correctly replaced. Redownload. (Not savegame compatible)
I get a clear database.log when I try it, now.


Yes I've heard this from several people. I'm personally playing with the edit version, which seems to still work.
I'm not sure it's really kosher to integrate it, but we can look into why the complete version has stopped working. I'll do that after Christmas.
Edit: I set the AI to autoplay with the complete version. It's at turn 112 and doing fine -- this is without UCS. With UCS it runs, but crashes as soon as I hover over a CS tooltip. There don't appear to be any database errors. It also crashes if I try open IGE.


Yes, the civ just hasn't been updated yet.
If a civ isn't compatible you can fix it yourself by going to the compatibility.sql and adding a line there for the civ,

INSERT INTO CD_Compatibility
    (CivilizationType, CultureType, UnitDialogueTag)
VALUES
    ('CIVILIZATION_HOOP_THROWER_TEHUELCHE', 'CULTURE_CD_PATAGONIAN', null),
     ('CIVILIZATION_LS_VIETNAM','CULTURE_JFD_FAR_EASTERN',    null),
...

or you can wait!

So I tried clearing the cache and redownloading, but unfortunately the issue appears to be unchanged. On the setup screen the number of city states is maxed out; trying to adjust it at all sets it to 0. Starting the game with or without adjusting it leads to an instant crash. Here are updated logs.

 

[hokath]

The error is

table MinorCivilizations has 16 columns but 15 values were supplied

Which then causes all city-states not to load in.
I see the CulDiv_Master.sql in JFD (1) does

ALTER TABLE MinorCivilizations ADD CultureType    text    default null;

So it could be that this messes with some xml file elsewhere.
This is with or without UCS? (stopwatch.log suggests this is VP + 5/6UC + some map script, is that right?)

 

[Bysonte]

Reached the modern era with Brazil and realized the embrapa buffs the resources substituded by the distillery.
Shouldn't the embrapa also buff the liquor or will have to change the resources it buffs?

 

[msw1]

The error is

table MinorCivilizations has 16 columns but 15 values were supplied

Which then causes all city-states not to load in.
I see the CulDiv_Master.sql in JFD (1) does

ALTER TABLE MinorCivilizations ADD CultureType    text    default null;

So it could be that this messes with some xml file elsewhere.
This is with or without UCS? (stopwatch.log suggests this is VP + 5/6UC + some map script, is that right?)

This is with VP +5/6UC + UCS. No other modmods.

 

[hokath]

I don't get the error you do. Is your UCS the most recent version?

@Bysonte good point, will add

 

[msw1]

Yeah UCS is the most recent version of 18.3. I tried redownloading to no avail. VP is also the most recent version of 5.0.1.

 

[Chinag]

Oh yeah, I'm also getting crashes with VP 5.0.1, 5/6 UC and UCS. Cache deleted and recently installed the recent versions of 5/6 UC and UCS v18.3

 

[msw1]

I think I figured it out. It's a load order issue. I made CD - (1) (Core) Utilities depend on UCS so that it loaded after, and I was able to start a game.

 

[ravennasea]

I think I figured it out. It's a load order issue. I made CD - (1) (Core) Utilities depend on UCS so that it loaded after, and I was able to start a game.

This fixed it for me

<Mod id="b2eb6d1e-1fcb-4a0d-829c-51691aa3183a" minversion="0" maxversion="999" title="Unique City-States" />

 

[cogwheel]

Regarding the Byzantines under Eastern European... not sure if this is a thematic fit? I see the additional components they have to be more Slavic than anything, while Byzantines seem to fit more under Classical.

Perhaps there should be an additional subculture for Hellenic civs? Of course, recognize that that would take additional effort to design and implement, so feel free to disregard if I sound too nitpicky

 

[hokath]

They're there because that's where they are in the original mod.
If we like, they could move to Classical. We had a discussion about this internally -- basically the Byzantine empire thought of itself as Roman, so... Classical

On the load order thing. I will add it to the version of (1) and hopefully this avoids problems for people in the future.
I think maybe it worked for me because it already reference Enlightenment Era, so with that active it got relegated in the load order?

 

[ravennasea]

They're there because that's where they are in the original mod.
If we like, they could move to Classical. We had a discussion about this internally -- basically the Byzantine empire thought of itself as Roman, so... Classical

On the load order thing. I will add it to the version of (1) and hopefully this avoids problems for people in the future.
I think maybe it worked for me because it already reference Enlightenment Era, so with that active it got relegated in the load order?

Maybe I am misunderstanding did you have EE reference this mod or are you saying you were playing with EE? I got this error while using EE

 

[hokath]

I was just playing with EE. Load order is a bit funny when not controlled, seems to vary wildly.
Glad we were able to fix it.

To clean things up, I've removed some unneeded references from the mod. It is now v2 (and not savegame compatible)
Changelog since last post
-------------
v1
-------------
...
-- Added Boarding Party 1 to Baghlah, so they carry over more on upgrade (lowered CS by 1 to compensate)
-- Grouped Sun Dancer and Izzat in the pedia, making them easier to find, and fixed a text error there
-- Sun Dance Site WLTKD turns dropped to 10 (was 20)
-- Undying Turtle defense boost increased to 15% (was 10%)
-- Added Liquor to the Embrapa list of buffed resources
-----------------------------
v 2
-----------------------------
-- Removed unnecessary dependencies copied over from another mod (no effect atm but housekeeping)
-- Halved the Pillage Gold on the Distillery (was 40)
-- Fixed Voivode flag

 

[Karlos_BR]

They're there because that's where they are in the original mod.
If we like, they could move to Classical. We had a discussion about this internally -- basically the Byzantine empire thought of itself as Roman, so... Classical

How about 'South European'? It's currently only used for Venice. The Stradiot were from the balkan region, Banco Giro however would need to be tuned/replaced by something less Venice-centric though...