-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Description
I'm interested in using this for 1D and 2D overlapping ranges but with RangeInclusive (specifically, for overlapping text ranges + overlapping 2D grid ranges). I managed to basically brute-force replace every Range with RangeInclusive in the library and got a local version of this working, except it doesn't appear to work right for 2D ranges (for this, instead I decided to use quadtree_rs) -- probably because you cannot implement Ord for 2D ranges reliably (instead, it forms a lattice-like ordering for disjoint ranges).
Anyways, I think it works for 1D RangeInclusive so I thought I'd raise up the issue. I'll try to see if I can get the code uploaded.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels