LoneGazebo / Community-Patch-DLL

Community Patch for Civilization V - Brave New World
Other
286 stars 158 forks source link

3/15 - Vassal tribute favors obsolete unique units #6529

Closed silentreply closed 4 years ago

silentreply commented 4 years ago

1. Mod version (i.e Date - 2/9): 3-15-2

2. Mod list (if using Vox Populi only, leave blank):

3. Error description: Vassal tribute seems heavily weighted towards unique units even when they are obsolete. In the below screenshot I entered information era, and all my vassals are in modern or atomic era. Minuteman are from at least two eras before any other gifted unit.

vassal tribute units information era

4. Steps to reproduce (optional):


Supporting information: Please note that you can attach .zip files by dragging-and-dropping them. If possible, zip up all supporting data and post that way.

  1. Log files (always attach your Logs folder, located at My Documents/My Games/Sid Meier's Civilization 5. Make sure you have enabled logging before experiencing an error! Go here to find out how: http://forums.civfanatics.com/showthread.php?t=487482):

  2. Save game (always attach a save that was made a turn before the error; located at My Documents/My Games/Sid Meier's Civilization 5/ModdedSaves):

  3. CvMiniDump.dmp file (attach if experiencing a game crash. Located at Program Files/Steam/steamapps/common/Sid Meier's Civilization V):

  4. Screenshots (optional):

RecursiveStar commented 4 years ago

This can also happen to UUs from City-States.

When prioritizing in this way, I think it should be based on whether the unit has any unique promotions that will remain on upgrade.

JamesNinelives commented 4 years ago

Hmm. I kinda like it this way in many cases, but it does seem reasonable to (as Recursive) mentions only do this for units that have promotions which persist on upgrades.