search

We-the-People-civ4col-mod / Mod

This is the repository where the mod resides.
89 stars 37 forks source link

Properly randomize leaders with randomize leader gameoption #220

Open Nightinggale opened 5 years ago

Nightinggale commented 5 years ago

I found a BTS forum thread with an interesting topic. Essentially random leaders will shuffle leaders rather than randomizing how each leader acts.

If we are to get proper random leaders, we should copy the leaderhead xml data into CvPlayer at game init and make the game read the cache in CvPlayer instead of looking up xml data. This will allow us to modify the cache in CvPlayer, meaning we can change stuff like how aggressive players are. We should add a new xml file, which contains the min and max of each of the random values, meaning it will be fully customizable from xml.

A nice benefit from this is that it will also make the code slightly faster, but I will be very surprised if you can tell the difference.

devolution79 commented 5 years ago

I think that this will make little difference unless we also implement #91. Currently the leaders only differ in their diplomatic parameters, not in actual behaviour (except the probability of declaring war) Some leaders are not really geared for warfare, some leaders are better off as peaceful builders (e.g. Champlain) while Washington is an obvious candidate for warmongering 😝