Opened 11 years ago

Last modified 6 years ago

#2192 new defect

Initial iberian walls are coliding with trees

Reported by: Josef Kufner Owned by:
Priority: Should Have Milestone: Backlog
Component: Core engine Keywords:
Cc: Patch:

Description (last modified by fabio)

Hello,

initial iberian walls are generated on top of trees and other resoources. When wall is destroyed, it is not possible to rebuild it, because the same place is occupied by a tree or something.

These resources should be moved a bit while placing the wall. It is better to have tree in middle of a rock, than in middle of the wall.

http://trac.wildfiregames.com/raw-attachment/ticket/2192/wall-beats-rock.jpg

Attachments (1)

wall-beats-rock.jpg (391.0 KB ) - added by Josef Kufner 11 years ago.

Download all attachments as: .zip

Change History (7)

by Josef Kufner, 11 years ago

Attachment: wall-beats-rock.jpg added

comment:1 by Adrián Chaves, 11 years ago

See also #1565. I am not sure whether this should be considered a duplicate or if both tickets can be kept, #1565 about the height issue and this one about the tree issue mentioned in the comments in #1565.

comment:2 by Josef Kufner, 11 years ago

Looks like very similar problem, but it is a bit different. I added link to #1565.

comment:3 by fabio, 11 years ago

Description: modified (diff)

comment:4 by historic_bruno, 11 years ago

We should really have one metaticket for rmgen's broken entity placement logic (and it's connected to #1589 as well). If you see chickens in CCs or mines like #1901, or trees overlapping mines, or buildings overlapping resources, it's all the same bug. The RMS basically just places on individual tiles and doesn't worry much about obstruction sizes (since it doesn't know them).

comment:5 by FeXoR, 10 years ago

historic_bruno:

There is: #1589

There's also a forum topic: http://www.wildfiregames.com/forum/index.php?showtopic=17052

Last edited 10 years ago by FeXoR (previous) (diff)

comment:6 by elexis, 6 years ago

In 20815:

Implement random map script playerbase function.
Unifies 54 variants of the rmgen playerbase code, fixes #4805.

Add retry loops to prevent collisions of starting resources, fixes #4600 and
resources placed outside of the map area, fixes #4796.

Lays the foundation to test for collisions with Iberian walls, refs #2192 and
allows to rearrange the starting resources on all random maps without being confronted with code.

Delete remains of misc.js, leaving the rmgen files sorted by logic, fixes #4804.
Concludes what was started in rP18816, rP19282, specifically the 82 rmgen commits starting rP20115, rP20301.
Uses vector algebra, refs #4845.
Removes many Math proxy calls, refs #4933.
Removes 35 unused elevation and 24 unused cliffRadius variables, demonstrating the copy&paste antipattern.
Reduce iberian-wall hardcoding to one line, refs #4940.

Note: See TracTickets for help on using tickets.