Bug Reports - Post Here!

Barra · Oct 22, 2014

Just want to ask if the CPP-Team knows about the little ressource icon display bug described here at bc1's EUI thread
descriped by Skodkim post 1172

and possibly answered by bc1 post 1179

http://forums.civfanatics.com/showthread.php?t=512263&page=59

I got the same bug and have cpp-dll + balancepatch + EUI + CSD + CivIVdiplo

I have checked the install instructions several times and all should be done fine installwise.

Maybe my lua.log helps:

Spoiler :

Gazebo · Oct 22, 2014

The Yield Icon Manager, if it is causing the problem, was designed by Whoward, and as such is outside of my domain of expertise.
G

skodkim · Oct 22, 2014

Hi G

Just thought I'd remind you of the bug from post 469 (since you never commented on it but made several later posts)

\Skodkim

Gazebo · Oct 22, 2014

skodkim said:
Hi G

Just thought I'd remind you of the bug from post 469 (since you never commented on it but made several later posts)

\Skodkim

Thanks, it had slipped through. I'll take a look. If it is a duplication error, it isn't the worst possible bug, but it still needs to be fixed. Some of the duplication errors are a result of whoward's individual mods overlapping with the CBP/CP, which is fine.
G

skodkim · Oct 23, 2014

Great.

I managed to tweak your files so that they at least do not cause error messages. I don't have the warmonger mini mod enabled by the way.

Another 'issue' that is somewhat related. When I meat AI players I can see a diplo message saying something like "early concerns about your warmongering". Thing is that I haven't even been to war with anyone but the barbarians yet...

\Skodkim

Gazebo · Oct 23, 2014

skodkim said:
Great.

I managed to tweak your files so that they at least do not cause error messages. I don't have the warmonger mini mod enabled by the way.

Another 'issue' that is somewhat related. When I meat AI players I can see a diplo message saying something like "early concerns about your warmongering". Thing is that I haven't even been to war with anyone but the barbarians yet...

\Skodkim

Sounds like an XML value is artificially inflating your warmonger value (this is probably related to whoward's warmonger change). I'll look into it.
G

ilteroi · Oct 25, 2014

hello again,

i just tried a little game with cp+cbp+csd: i was third to adopt an ideology - chose the last free one - and bam! : -97 happiness ... are you aware of this? it seems to be calculated 1:1 from the number of citizens. i don't think that can be right

UPDATE: debugger says

if (MOD_DIPLOMACY_CITYSTATES) {
iPerCityUnhappy = GC.getIDEOLOGY_PER_CITY_UNHAPPY();
iUnhappyPerXPop = GC.getIDEOLOGY_POP_PER_UNHAPPY();
}

returns 0 for both variables.

but in csd/AddDefines.xml it says:

INSERT INTO Defines (
Name, Value)
SELECT 'IDEOLOGY_PER_CITY_UNHAPPY', '2';

INSERT INTO Defines (
Name, Value)
SELECT 'IDEOLOGY_POP_PER_UNHAPPY', '6';

so i'm confused. is this a real bug or an issue with my setup?

in any case, if it were working as intended, i'd still have 16 unhappiness - from a minimal dissatisfaction value of 1.

in general, i propose to get rid of the hardcoded thresholds 3 and 5 in ComputePublicOpinionUnhappiness() and use a linear scaling instead.

Gazebo · Oct 27, 2014

ilteroi said:
hello again,

i just tried a little game with cp+cbp+csd: i was third to adopt an ideology - chose the last free one - and bam! : -97 happiness ... are you aware of this? it seems to be calculated 1:1 from the number of citizens. i don't think that can be right

UPDATE: debugger says

if (MOD_DIPLOMACY_CITYSTATES) {
iPerCityUnhappy = GC.getIDEOLOGY_PER_CITY_UNHAPPY();
iUnhappyPerXPop = GC.getIDEOLOGY_POP_PER_UNHAPPY();
}

returns 0 for both variables.

but in csd/AddDefines.xml it says:

INSERT INTO Defines (
Name, Value)
SELECT 'IDEOLOGY_PER_CITY_UNHAPPY', '2';

INSERT INTO Defines (
Name, Value)
SELECT 'IDEOLOGY_POP_PER_UNHAPPY', '6';

so i'm confused. is this a real bug or an issue with my setup?

in any case, if it were working as intended, i'd still have 16 unhappiness - from a minimal dissatisfaction value of 1.

in general, i propose to get rid of the hardcoded thresholds 3 and 5 in ComputePublicOpinionUnhappiness() and use a linear scaling instead.

Hmm...I'll look at this and check into it (sitting down today, for the first time in awhile, with an eye to work on the mods). Sounds like those values simply aren't being passed into the database correctly.

I managed to tweak your files so that they at least do not cause error messages. I don't have the warmonger mini mod enabled by the way.

How did you edit the files?
G

skodkim · Oct 27, 2014

Gazebo said:
How did you edit the files?
G

Attached both original files from 63.4.1 and modified files in the post. Knock yourself out

\Skodkim

skodkim · Oct 28, 2014

Smal typo in v 63.4.1.
Comitted is spelt "comittedd" in NewText.xml

Spoiler :

\Skodkim

Gazebo · Oct 28, 2014

skodkim said:
Smal typo in v 63.4.1.
Comitted is spelt "comittedd" in NewText.xml

Spoiler :

<Text>{1_SpyRank} {2_SpyName} has successfully committedd an assassination in {3_CityName}, delaying [ICON_GREAT_PEOPLE] Great Person progress by {4_Num}!</Text>

\Skodkim

Awesome, fixed. Also, how do you like the extra spy actions in the CBP? They aren't as fancy as BE's, of course, but that's to be expected.
G

skodkim · Oct 28, 2014

Gazebo said:
Awesome, fixed. Also, how do you like the extra spy actions in the CBP? They aren't as fancy as BE's, of course, but that's to be expected.
G

Sorry for the missing feedback on those :rolleyes:

I like them! It's not as though the effects are disasterous (which they shouldn't be!!) but spying in the normal game is simply boring so anything that can add a bit of flavor to it is welcome! :goodjob:

Have you had a look at the warmonger bug yet (no pressure, just plain interest)?

\Skodkim

Aussie_Lurker · Oct 30, 2014

Not sure if anyone has mentioned this but, with the most current version of Community Patch, there seems to be an issue with randomly gaining tourism. It's only +1 to +2 so far, but still thought it was worth mentioning!

Aussie_Lurker.

Gazebo · Oct 30, 2014

Aussie_Lurker said:
Not sure if anyone has mentioned this but, with the most current version of Community Patch, there seems to be an issue with randomly gaining tourism. It's only +1 to +2 so far, but still thought it was worth mentioning!

Aussie_Lurker.

This is probably a problem with the fact that the current version doesn't include the changes from the latest dll (civ is patched beyond the CP). This will be fixed in the next version.
G

skodkim · Oct 31, 2014

Hi G

Just downloaded the new mod combo (Versions 64/11/25), cleared cache and launched the game. No other mods involved and haven't altered ini files (it should check for CSD automatically now, right?).

Using this "default" combo I get errors in database.log. I reported the one regarding CultureOverview.xml for version 63.4.1 earlier on. See the attached files.

Spoiler :

Otherwise good job - can't wait to try it out ! :goodjob:

\Skodkim

Gazebo · Oct 31, 2014

skodkim said:
Hi G

Just downloaded the new mod combo (Versions 64/11/25), cleared cache and launched the game. No other mods involved and haven't altered ini files (it should check for CSD automatically now, right?).

Using this "default" combo I get errors in database.log. I reported the one regarding CultureOverview.xml for version 63.4.1 earlier on. See the attached files.

Spoiler :

[458.689] table Building_GrowthExtraYield already exists
[458.689] In XMLSerializer while building table Building_GrowthExtraYield from file Core Files/Core Tables/CSDTable.xml.
[460.873] Database::XMLSerializer (LUA/Active/CultureOverview.xml): 'Row' or 'Delete' expected, got 'Box'.
[461.716] column Name is not unique
[461.716] column Name is not unique
[461.716] columns Language, Tag are not unique
[461.716] columns Language, Tag are not unique
[469.204]

Otherwise good job - can't wait to try it out !

\Skodkim

Those errors are normal (though not the language/tag ones - I think those are related to the PNM mod) - I duplicated the CSD 'growth yield table' in the CP so that it would definitely work whether or not a player was using CSD with or without the CP.

Let me know how it goes!
G

greyhoundgames · Oct 31, 2014

Saw a situation where 2 barbarians were milling about around an AI city at about 10% health. They had pillaged everything but the ai and her archer(eliz) seemed unwilling to shoot at them to finish them off. I was trying to get a prophet in so I watched for quite a few turns. I am pretty sure i saw the foritfy animation play on the archer on the computer's turn. No other mods running except for the EUI mod.

Aussie_Lurker · Oct 31, 2014

Community Balance Patch v11 seems to be creating some issues with cities. When you found a city, you get a couple of spy icons appear *and* you get the ranged combat option coming up too. I take it this *isn't* meant to happen?

Aussie.

lockstep · Oct 31, 2014

Tried to use the newest versions (65 resp. 11) of CP/CBP together with EUI 1.23b. As per various instructions, I removed YieldIconManager.lua and YieldIconManager.xml from CP plus CityBannerManager.lua, CityStateStatusHelper.lua and CityView.lua from CBP. Result: CP/CBP without EUI seem to work fine, as does CP only together with EUI; CP/CBP together with EUI result in a crash to desktop as soon as I try to found my first city. Various log files plus a crash dump file are attached.

Aussie_Lurker · Oct 31, 2014

Another issue that I have found. The diplomacy screen is a total mess. When you go into the discuss section, you get the same option repeated about a dozen times....also, none of the discuss options actually work when you click on them :-(.

Aussie.

Bug Reports - Post Here!

Warlord

Lord of the Community Patch

Deity

Lord of the Community Patch

Deity

Lord of the Community Patch

Prince

Lord of the Community Patch

Deity

Attachments

Deity

Lord of the Community Patch

Deity

Deity

Lord of the Community Patch

Deity

Attachments

Lord of the Community Patch

Chieftain

Deity

Prince

Attachments

Deity

Similar threads