Jump to content

[re-report] Wall Placement Regression


chaosmonkey
  • Fixed

Walls can no longer be placed adjacent to glowcaps if the walls are placed on the wall grid. I'm including before/after photos to demonstrate.

First image shows that a wall may be placed directly next to the glowcap on the wall grid.

production_wall_placement.jpg.92ca0f67c3869f31853335a44799f751.jpg

This next image is the current beta patch which does not allow placement directly next to the glowcap. (note the blacked out placement options).

beta_wall_placement.jpg.9a8f23f7e1e1fb37a12157e6458a47db.jpg

Looking at the `IsNearOther` logic for deploy_smart_radius,

	elseif other.deploy_smart_radius then
		min_spacing = other.deploy_smart_radius + (min_spacing or math.sqrt(min_spacing_sq)) / 2
		min_spacing_sq = min_spacing * min_spacing

Wall spacing is going to increase in any case where the `min_spacing` argument is > other.deploy_smart_radius. I think for backwards compatibility, the deploy smart spacing should be doing `min_spacing = math.min(min_spacing or math.sqrt(min_spacing_rq), other.deploy_smart_radius)`.

One other alternative, the glowcap base is pretty narrow. Maybe glowcap min_spacing could be reduced to 1 and smart spacing to 0.5? 


Steps to Reproduce

Place a glowcap on the wall grid

Try and place a wall next to it axis aligned.




User Feedback


A developer has marked this issue as fixed. This means that the issue has been addressed in the current development build and will likely be in the next update.


Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×
  • Create New...