I believe this needs to be non-const
auto &&values, some standard views have non-const begin()/end(). Also ranges::input_range is probably a bit better than just range.