Jump to content

Wall placement regression

chaosmonkey

By chaosmonkey

    • Pending

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

 

Steps to Reproduce

Place a glowcap on the wall grid.

Try and build a wall directly next to it.

Go to bugs

User Feedback


chaosmonkey

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 0.5 smart spacing? Seems reasonable just off appearances.

Share this comment

Link to comment
Share on other sites


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...