Thanks, it's an interesting feature! Do you think you'll try a mutable version as well?

And yes, we'll want to wait for the API to stabilize in the standard library so we can match it.

That should be the mutable version.

Just for reference, that slice_group_by feature is tracked in rust-lang/rust#80552.

I'm interested in this, but I found a bug in it, the following call:

let data = vec![1, 1, 2, 2, 2, 2, 2, 3, 3, 3, 4, 4];
let groups: Vec<_> = data.par_group_by(|&a, &b| a == b).collect::<Vec<_>>();

println!("{:?}", groups);

will result in 5 groups instead of the expected 4, these groups are:

[[1, 1], [2, 2, 2, 2, 2], [3], [3, 3], [4, 4]]

I think that fixed it

I think that fixed it

It did, nice job!

FYI, that stabilization just landed with a rename to chunk_by / chunk_by_mut:
rust-lang/rust#117678

I wonder if this can be refactored to use the internal SplitProducer infrastructure,

I didn't find a good way to shoehorn this into SplitProducer, per se, but I've now made the implementation quite similar to that, and a test asserts that we get exactly len() - 1 comparisons.

Thanks @jakeKonrad! Hope you don't mind that I wrapped this up myself.