Opened 7 years ago
Closed 6 years ago
#4830 closed defect (fixed)
Pigs inside the CC on Deep Forest
Reported by: | elexis | Owned by: | elexis |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Alpha 23 |
Component: | Maps | Keywords: | |
Cc: | Patch: |
Description
The map Deep Forest places pigs near the civic center - sometimes inside the civic center.
Besides that the pigs can't be gathered from, units trying to reach them won't show up as idle.
Reproducible with the attached r20327 replay that can be started for alpha 22 as well (from command line or editing the version number).
Attachments (1)
Change History (4)
by , 7 years ago
Attachment: | commands.txt added |
---|
comment:1 by , 6 years ago
comment:2 by , 6 years ago
Keywords: | simple removed |
---|---|
Milestone: | Backlog → Alpha 23 |
Note:
See TracTickets
for help on using tickets.
Not so easy to solve.
The idea of Deep Forest is to be fancy and place entities using terrain painting rather than specific entity placement.
The city patch consists of three terrains. Pigs and chicken are part of the middle terrain. The LayeredPainter receives the outer width first, then the middle width and the inner width is the rest. This means we can't specify the inner width to ensure a minimum distance.
This means if the Terrain based entity placement approach is kept for the city patch, it will be either very easy to reintroduce that bug when only messing with the numbers, or we have to add code to remove entities again after having them placed already.
And that was only the consideration of the civic center. The pigs might just as well be stuck inside metal mines, which completely invalidates any radius-based testing.
So it's much safer to just place entities directly with avoidClasses constraints.