Opened 6 years ago

Last modified 5 years ago

#5118 new defect

Walls not build if wall tower foundations overlap

Reported by: elexis Owned by:
Priority: Should Have Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch:

Description (last modified by elexis)

As noticed by temple spectating a Hellas game with Emperior, it can occur that wall pieces are not being built following Phab:D21 if the obstructions of the wall tower foundations overlap.

https://trac.wildfiregames.com/raw-attachment/ticket/5118/M7btW6C.jpg

This regression must be fixed before the release. The easiest way to accomplish that is making structures not block construction as proposed in Phab:D1445 (and the case in a22).

But there is also the question if this ideal. It'd be better if foundations cannot overlap (which was an assumption of Phab:D21.)

There is a patch for the templatesanalyzer detecting incompatible lengths at Phab:D1440. Phab:D1439 changes wall tower lengths to match the obstruction size.

Attachments (1)

M7btW6C.jpg (282.6 KB ) - added by elexis 6 years ago.

Download all attachments as: .zip

Change History (6)

by elexis, 6 years ago

Attachment: M7btW6C.jpg added

comment:1 by elexis, 6 years ago

Description: modified (diff)

comment:2 by elexis, 6 years ago

In Walls.js added by r11760:

TODO: make sure intermediate towers are spaced out far enough for their obstructions to not overlap, implying that tower's wallpiece lengths should be > their obstruction width, which is undesirable because it prevents towers with wide bases

comment:3 by temple, 6 years ago

In 21702:

Make structures not block construction

Differential Revision: https://code.wildfiregames.com/D1445
Reviewed by: elexis
Refs: #5118

comment:4 by elexis, 6 years ago

Milestone: Alpha 23Backlog
Priority: Release BlockerShould Have

Thanks for fixing the regression

comment:5 by Imarok, 5 years ago

Component: UI & SimulationSimulation

Move tickets to Simulation as UI & Simulation got some sub components.

Note: See TracTickets for help on using tickets.