bug in `intersect`

[ericphanson]

bug in `intersect`

[ericphanson]

Oh! Ok. That's pretty confusing. Can we do a breaking release & drop the deprecation?

[iamed2]

That call doesn't actually take the intersection of the two intervals.

To do that:

julia> intersect(Interval{Int, Closed, Open}(1,3), Interval{Int, Closed, Open}(3,5))
Interval{Int64, Open, Open}(0, 0)

Intersect of one argument is an identity operation. For some reason Julia dispatches intersect(s) = union(s), so the misleading union method on arrays of intervals that is currently deprecated takes over. Once that deprecated method is removed, this call will return an array containing the same intervals in the same order.

[iamed2]

Seems reasonable, although it might be good to address any invalidations that still exist at the same time.