Project

General

Profile

Bug #695979

civ2civ3: Desert with river should require irrigation to get extra food

Added by David Fernandez (bard) 15 days ago. Updated 13 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Rulesets
Target version:
Start date:
Due date:
% Done:

0%


Description

Desert with river requires irrigation to get extra food.
Bug reported by Jacob Nevins in Feature #681980.

26-civ2civ3-Extra-food-on-Deserts-with-river-requires-irrigation.patch View (2.45 KB) David Fernandez (bard), 2017-09-09 03:13 PM

m-30-civ2civ3-nile-floods-fix.patch View (5.21 KB) Jacob Nevins, 2017-09-09 03:51 PM

26-civ2civ3-nile-floods-fix.patch View (5.22 KB) Jacob Nevins, 2017-09-09 03:52 PM


Related issues

Related to Freeciv - Feature #681980: civ2civ3: use Mining_Pct for oil well, and Irrigation_Pct for desert flood. Closed
Blocks Freeciv - Task #694704: civ2civ3: introduce new features for 2.6 Closed

History

#1 Updated by Jacob Nevins 15 days ago

  • Category set to Rulesets
  • Status changed from New to In Progress
  • Assignee set to Jacob Nevins

I already have a patch for this.

#2 Updated by Jacob Nevins 15 days ago

  • Subject changed from Desert with river requires irrigation to get extra food to civ2civ3: Desert with river should require irrigation to get extra food

#3 Updated by Marko Lindqvist 15 days ago

  • Blocks Task #694704: civ2civ3: introduce new features for 2.6 added

#4 Updated by Jacob Nevins 15 days ago

The patch needs to be a little more complicated than that in order to do the right thing on city centres.

#5 Updated by Jacob Nevins 15 days ago

  • Related to Feature #681980: civ2civ3: use Mining_Pct for oil well, and Irrigation_Pct for desert flood. added

#6 Updated by David Fernandez (bard) 15 days ago

I always forget about the city center... sorry.
In that case, why not simply revert to use Output_Inc_Tile. I suggested Irrigation_Pct because I thought it would be simpler to implement and maintain, but that doesn't seem to be case.

Anyway, I don't want to be a cause of extra work for you, I let you implement it as you see better.

#7 Updated by Jacob Nevins 13 days ago

  • Status changed from Resolved to Closed

why not simply revert to use Output_Inc_Tile

Oops, I didn't think of that :)

Will probably get back and do this at some point; while it doesn't change the rules, a simpler effect representation might make it easier for the AI to understand the rules, I guess.

Also available in: Atom PDF