tweak check

Author: ericphanson

src/interop.jl

@@ -79,24 +79,26 @@ function stop_index_from_time(sample_rate, interval::Interval{Nanosecond,Closed,
     # `time_from_index(sample_rate, last_index + 1)` gives us the _start_ of the next sample
     # we subtract 1 ns to get the (inclusive) _end_ of the span associated to this sample
     end_of_span_time = time_from_index(sample_rate, last_index + 1) - Nanosecond(1)
-    # if this end isn't fully included in the interval, then we need to go back one
-    if !(end_of_span_time in interval)
+    # if this end isn't fully included in the interval, then we need to go back one.
+    # Note: we don't use `end_of_span_time in interval`, since that could occur if we are before
+    # the _start_ of the interval, rather than past the end of it.
+    if !(end_of_span_time <= last(interval))
         @debug "Decrementing last index to fully fit within span"
         last_index -= 1
     end
 
     # We should never need to decrement twice, but we will assert this
     end_of_span_time = time_from_index(sample_rate, last_index + 1) - Nanosecond(1)
-    if !(end_of_span_time in interval)
+    if !(end_of_span_time <= last(interval))
         msg = """
         [AlignedSpans] Unexpected error in `stop_index_from_time` with `RoundFullyContainedSampleSpans`:
 
         - `sample_rate = $(sample_rate)`
         - `interval = $(interval)`
         - `mode = $(mode)`
         - `end_of_span_time = $(end_of_span_time)`
-        - `interval = $(interval)`
-        - Expected `end_of_span_time in interval`
+        - `last(interval) = $(last(interval))`
+        - Expected `end_of_span_time <= last(interval)`
 
         Please file an issue on AlignedSpans.jl: https://github.com/beacon-biosignals/AlignedSpans.jl/issues/new
         """