Rework Atlantean Crystal Mine

Bug #1753248 reported by Teppo Mäenpää
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Wishlist
Unassigned

Bug Description

Atlantean marble mine works poorly when depleted. More poorly than the other tribes' mines, that is.

Ways to improve:
- Each work step only takes action, if the main output is needed
- The first mine-produce -cycle of each program produces at least one ware of needed type.

Not producing the needed ware right away lowers the percentage of success from 0.05 to 0.05^n, where n is the number of step that produces the needed ware. This is unfair, a five-percent success rate is big enough handicap.

See Forum ( https://wl.widelands.org/forum/topic/4212 ) for discussions.

Tags: tribes

Related branches

Teppo Mäenpää (kxq)
Changed in widelands:
importance: Undecided → Wishlist
assignee: nobody → Teppo Mäenpää (kxq)
Revision history for this message
Teppo Mäenpää (kxq) wrote :

I attach a simple save game with three Crystal mines and 1000 fooditems for easy testing.

Load save game, and observer the accumulation of granite and quartz. The difference between attached branch and current trunk describes the difference quite clearly.

Revision history for this message
Teppo Mäenpää (kxq) wrote :
Revision history for this message
Teppo Mäenpää (kxq) wrote :
Revision history for this message
GunChleoc (gunchleoc) wrote :

Those graphs look good ;)

tags: added: tribes
Revision history for this message
Teppo Mäenpää (kxq) wrote :

Now the granite-cycle produces
granite:2
granite:2
granite
quartz

One could boost the empty mines a bit more, for example like this:
granite:3
granite
granite
quartz

This way, the behavior with non-depleted mines is the same (unless micromanaging), and mountain releases same amount of resources before depleting, but the empty-mine would be boosted a bit wrt others. I think that the mines would still not be too good, but this is subjective.

The branch, as it is now, fixes a real bug and should go to trunk in my opinion.

I am less sure if changes like drafted above should be done too.

Revision history for this message
Teppo Mäenpää (kxq) wrote :
GunChleoc (gunchleoc)
Changed in widelands:
milestone: none → build20-rc1
status: New → Fix Committed
assignee: Teppo Mäenpää (kxq) → nobody
Revision history for this message
Teppo Mäenpää (kxq) wrote :

Thanks.

Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build20-rc1

Changed in widelands:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.