Closed iamed2 closed 7 years ago
@mbauman @ajkeller34 review/merge?
Good grief, how has the Mac Travis CI been queued for 6 hours? I tested your branch on my Mac and it passed, so I'll just merge this since it also passed on Linux.
Travis really dropped the ball on maintenance of their Mac infrastructure: https://www.traviscistatus.com/
Previously, every unknown eltype (not a subtype of Number, AbstractTime, etc.) would require a new
axistrait
definition, or wrapping in a SortedVector or CategoricalVector type. Now, the default behaviour for indexing using an AbstractArray axis is Categorical, which requires only equality and notisless
.I also made arrays of Value types have the axistrait of either Categorical or Dimensional depending on the type they wrap, which maintains existing tested behaviour.