Closed jgpruitt closed 1 year ago
One thing I was thinking that maybe we should have one of the filters be inclusive while the other not, like
from < time <= until
so that it's easier to iterate over the time ranges.WDYT?
Yes, definitely. IMHO it should be from <= time < until
.
Agreed. In my experience, lower inclusive upper exclusive has been the default.
This is ready for review @JamesGuthrie
And @jgpruitt can you double check the changes to the query?
One thing I was thinking that maybe we should have one of the filters be inclusive while the other not, like
from < time <= until
so that it's easier to iterate over the time ranges.WDYT?