Box inclusive contains about euclid HOT 7 CLOSED

Why not. We haven't needed something like this so far but it's not a big burden to add and maintain. Is it something you need in your projects?

from euclid.

yes, I'm using an inclusive contains. Basically to detect if the cursor is on top of a region, but the cursor snaps to a grid.

I've since impl'd this in a trait extension and it works really well, but perhaps others may find this useful as well

from euclid.

I have written something similar, except that instead of just creating an inclusive version of contains I have associated functions:

pub trait BoxInclusive {
  type Point;
  
  fn new_inclusive(min: Self::Point, max: Self::Point) -> Self;
  
  fn from_points_inclusive<I>(points: I) -> Self
  where
      I: IntoIterator,
      I::Item: Borrow<Self::Point>;
}

These simply create a normal box, then extend the max point by one in every dimension so that the max point itself is contained.

EDIT: I just submitted a PR for something unrelated, so I already have the project forked. I'd be happy to issue another PR for these inclusive box methods if there is interest from the maintainers in adding them to the box types directly.

from euclid.

I'm happy to take a PR for this if you want to make one.

from euclid.

These simply create a normal box, then extend the max point by one in every dimension so that the max point itself is contained.

This sounds very much like what I did, but wouldn't it be better to just have contains_inclusive functions with the > replaced with >= from performance and aesthetics considerations?

from euclid.

I have a need to create and use inclusive boxes without even necessarily ever using the contains method. Whatever approach the maintainers prefer is what we should go with.

from euclid.

Adding contains_inclusive sounds good to me.

from euclid.