Thalassicus / cep-bnw

Civ V Communitas Expansion Pack
32 stars 22 forks source link

CEAI_LeaderPersonalities & CEAI_LeaderPersonalities #185

Closed GrantSP closed 10 years ago

GrantSP commented 10 years ago

These exist in both CAT & CEG with some minor differences, probably errors from one being changed and not the other. e.g. Enrico Dandolo has differing values in them.

e.g. The formatting varies but the data is essentially the same in both Personalities files.

The question is: Is there any essential value for having these in both? If there are files that make changes in CAT based on these files being in CAT, can't we just remove them from CEG? Conversely, if CAT makes no use of them they would be better positioned in CEG, where the bulk of the changes to Leaders are made.

I haven't had time to check the references to these values by either module yet, if anyone has already done so, can you give a heads up?

GrantSP commented 10 years ago

Likewise the text files I have been working with. AI.sql Core.sql CSD.sql Promotions.sql etc.

If we can have a definitive decision on which files need to go where then the duplication errors will be reduced.

GrantSP commented 10 years ago

e.g. Eric just made some minor changes to the text for the Smith. Those changes were made before by someone but in the CAT text not the CEG text. Clearly then Core.sql needs to be in CEG, not sure why we need it in CAT.

GrantSP commented 10 years ago

Duh!!!

I just realized I have been working in the wrong directory. Stoopid

The entire GitHub directory has ALL the files but the actual mod directory has just the needed.

Idiot. Time for a break.