[BETA] Incorrect yield prediction

[bc1]

When looking at setter worker actions which remove a jungle within the area of a city with a university, predicted yield change forgets to mention the -2 science

 

[Camikaze]

Is this true for the fall patch public beta (1.0.3.124)?

 

[bc1]

Yes it is also true for the beta 1.0.3.124.
Furthermore, with the beta there is a further bug: mouseover says "x turns to Construct a Trading Post 1 turn to Create Fishing Boats" (this is with an old game save)

 

[Camikaze]

Even though this is something that would be fairly easily reproducible, could you please post a save in which the problem is demonstrated? A screenshot of the issue would be awesome as well!

Moderator Action: [BETA] prefix added.

 

[bc1]

There's no need, the bug can easily be found in CvPlot.cpp: the function CvPlot::CalculateImprovementYieldChange does not seem to check whether the improvement build removes the feature and whether this impacts yield

 

[dshirk]

A save would be handy so the engineers do not need to set-up the situation, and QA can easily verify any fix that we apply. Thanks!

 

[bc1]

Please find attached (there are a bunch of workers around Arthus to try out)
Cheers

 

[dshirk]

Thanks much, taking a look.