Region3s can have negative size very easily

[Sharksie]

> print(Region3.new(Vector3.new(1,1,1),Vector3.new(-3,-3,-3)).Size) -4, -4, -4

Just by supplying the lower vector second you can give a region3 negative size and then it won’t be able to detect anything with FindPartsInRegion3. It really shouldn’t even be possible for size to be negative.

[Maelstronomer]

It would be nice if the region3 constructor did this automatically:

local start_point = Vector3.new(5, 8, 5) local end_point = Vector3.new(8, 5, 8) Region3.new( Vector3.new( math.min(start_point.X, end_point.X), math.min(start_point.Y, end_point.Y), math.min(start_point.Z, end_point.Z) ), Vector3.new( math.max(start_point.X, end_point.X), math.max(start_point.Y, end_point.Y), math.max(start_point.Z, end_point.Z) ) )

[woot3 (Woot)]

[quote] It would be nice if the region3 constructor did this automatically:
[/quote]
While I agree this is useful, it means that it’s being changed from what you actually set it to which is quite bad practice. Instead, it should be able to account for a negative region as Ethan says.

[sleitnick]

Bumping because this is still relevant and causing issues:

Region3 Size is becoming negative?

I have created two bricks with the size of (0.05, 0.05, 0.05) at the top and bottom corners of a brick and I wanted to create a Region3 to check if the region is empty but for some reason it keeps saying that the region isEmpty when it should be printing false. After trying to figure out what had been going wrong I checked the size of the Region and it was printing the Y and Z axis as negatives. I am assuming that this is the issue causing my problem but I cannot think of what’s causing this is…

Region3s should not be able to be negative. At the very least, negative axis values should be clamped to 0. Considering nothing will ever rely on a negative Region3 (or at least shouldn’t), making this change should not break anything.

[EchoReaper]

Maelstronomer’s post sounds like a good idea. I should be able to give a region3 constructor any two corner points and construct a region from that. Currently behavior is confusing for new developers and annoying to work around for everyone else (have to manually convert to min/max)