Don't allow AdvancedSubtensor and AdvancedIncSubtensor non-consecutive advanced indices

### Description

Make the constructors shuffle it so we never have to worry about this case in the rewrites (happen all the time, and we almost always bail out not to worry about complexity). Make the AdvancedOp refuse already at init looking at idx_list