Make typing of `src_gids` and `target_gids` consistent

[ntolley]

At the moment these variables are passed around the code as a variety of types,, namely sets, lists, ranges. It might be worth looking at this globally and see if we can use a consistent type throughout (probably just set objects like in #642 )