[Feature]: Deprecate +contrib

[ehennestad]

What would you like to see added to MatNWB?

Deprecate +contrib namespace and/or move it to separate repository like matnwb-third-party

Is your feature request related to a problem?

No response

What solution would you like?

See above

Do you have any interest in helping implement the feature?

Yes.

Code of Conduct

• I agree to follow this project's Code of Conduct
• Have you ensured this change was not already requested?

[ehennestad]

@bendichter , What do you think?

[bendichter]

I'd be fine with that as long as there are docs that point to it. I haven't touched these functions in years and they are pretty basic. I'm hesitant to go too far down this road though since it seems like that would just end duplicating a lot of the effort in NeuroConv, but it's nice to have something that can demonstrate basic ideas of reading and writing data.

[ehennestad]

The main motivation would be to deprecate it from matnwb, but not to delete it completely as it might be still useful for people.

since it seems like that would just end duplicating a lot of the effort in NeuroConv

Agreed, it can even be explicitly stated that NeuroConv is the recommended library for conversion of neurodata.

[bendichter]

Sounds good!