Bugs item #3187552, was opened at 2011-02-20 15:59
Message generated for change (Tracker Item Submitted) made by jameskuyper
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=435578&aid=3187552&group_id=43225
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Client Gameplay
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: James Kuyper (jameskuyper)
Assigned to: Nobody/Anonymous (nobody)
Summary: Efficiency bonus and bell production
Initial Comment:
I'm using freecol 0.9.2. I've got a game with Thomas Jefferson, and two colonies with Newspapers and one Elder statesmen each. In the older colony, I'm getting a +1 efficiency bonus, while in the newer colony I'm not getting any bonus, but it can incur an efficiency penalty of -1 by adding one more member to the colony. Using the right click to examine the production modifiers, I find that with a +1 bonus, an Elder Statesman produces (3*2*(1+0.50)+1)*(1+1.0) =20 Bells, where the 3 is the base production, the 2 is due to the Elder Statesman, the 0.50 is due to Thomas Jefferson, the +1 is due to the efficiency bonus, and the 1.0 is due to the Newspaper.
When there is no efficiency bonus, the Elder Statesman only produces 15 Bells. Examining the production modifiers screen, I've concluded that this is being calculated as 3*2*(1+0.5+1.0)=15.
Finally, with a negative modifier, the Elder Statesman actually produces 16, which I conclude is being calculated as (3*2*(1+0.5)-1)*(1+1.0).
In other words, the Thomas Jefferson and Newspaper effects are additive when there's no efficiency bonus or penalty, but are multiplicative when there is one, with the bonus being applied after the Thomas Jefferson effect, but before the Newspaper effect. This means that bell production is more efficient when there's an efficiency penalty than it is with neither a bonus nor a penalty This seems inconsistent, but the description in the documentation aren't specific enough to be sure that this is unintentional.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=435578&aid=3187552&group_id=43225
|