check AbstractVector{Int} index sorted order

rafaqz · Nov 3, 2024 · 93ad378 · 93ad378
1 parent 6e96e27
commit 93ad378
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/Lookups/indexing.jl b/src/Lookups/indexing.jl
@@ -8,7 +8,10 @@ for f in (:getindex, :view, :dotview)
         @propagate_inbounds Base.$f(l::Lookup, i::Union{AbstractVector,Colon}) = 
             rebuild(l; data=Base.$f(parent(l), i))
         @propagate_inbounds function Base.$f(l::Union{Sampled,Categorical}, i::AbstractVector{Int})
-            rebuild(l; data=Base.$f(parent(l), i), order=Unordered())
+            if isordered(l)
+                issorted(i) || throw(ArgumentError("For `ForwardOrdered` or `ReverseOrdered` lookups, indices of `AbstractVector{Int}` must be in ascending order"))
+            end
+            rebuild(l; data=Base.$f(parent(l), i))
         end
         # Selector gets processed with `selectindices`
         @propagate_inbounds Base.$f(l::Lookup, i::SelectorOrInterval) = Base.$f(l, selectindices(l, i))