Alessio Buccino

Made a PR with a doc clarification on the inheritance of ms_before/after. Let's close this one https://github.com/SpikeInterface/spikeinterface/pull/3663

We also have the `has_extension` function. Probably better to use it instead of `if analyzer.get_extension("...") is not None`

LGTM!

> The current global default appears to be `n_jobs=1` so we discussed whether that should be changed for people who don't know to set `n_jobs` so let me know if...

I agree. Maybe a warning at the `fix_job_kwargs` level that "warns" that using `n_jobs=1` is not optimal?

@samuelgarcia suggested a global flag `global_kwargs_set` that by session is set to True if the user calls the `set_global_job_kwargs`. I think it's a good idea :)

@zm711 I made the suggested changes. Now the `fix_job_kwargs` warns if `n_jobs=1` and it hasn't been set (either globally, or locally). Also extended a bit the job_kwargs documentation.

Thanks! I'll give it a stab to parallelize it soon

I think that only maintainers can push! I'll add the suggested test :)

> I had a couple of questions: > > * The `motion_lim` is documented as `Tuple` or `None` but it actually expects an `int`. I guess this could be changed...