[feat-core] remove all classes (except for base) from init

[alejoe91]

Since we encourage a functional approach, I suggest we remove all classes (e.g. for preprocessing, sorters, etc.) from the __init__ file. This will make package exploration with tabs easier

@samuelgarcia?

[samuelgarcia]

No sure.
For notebook,
rec, sorting = si.read_mearec() si the way to go for me.
but in some pipeline having rec = si.MEArecRecording is also cool.
I think this could break lot of code for few benefit.
Lets think about this a bot more.

[alejoe91]

why? I think the functional apporach is always easier to understand and to write (except for very advanced use cases, like CollisionGTComparison etc.)

[alejoe91]

I haven't used a class in ages since the new API...

[h-mayorquin]

Something to add to this debate is that the current form hampers tab discoverability by duplicating names:

I personally prefer classess but I would rather expose only one way of doing things.

[zm711]

This is old. Should we close just because the default is to do nothing? (Though I agree with @alejoe91 for this one!)

[h-mayorquin]

I also changed my mind to @alejoe91 position. I personally use the classess but I also tend to do the full import path so this does not affect me and it will reduce the number I wish there was a way of getting directly from the lower case version to the class directly though.

The concern of breaking code is valid though...

[zm711]

@alejoe91 we discussed this and now Sam has converted so as long as you're okay with it I'll remove the classes from the init so that we only auto expose functions for users and devs can import the classes themselves by direct import.

[h-mayorquin]

Let's close this once we remove the class name.

[zm711]

So I've been looking into this a bit more and it seems a little more involved. Many of our inits use the class attributes so we import the class to get those attributes (like names etc). So let's bring this back up at the next maintenance meeting since Sam gave his okay we can briefly talk strategy for this.

[alejoe91]

fixed by #3898