search

mne-tools / mne-nirs

Process Near-Infrared Spectroscopy Data in MNE
https://mne.tools/mne-nirs/
BSD 3-Clause "New" or "Revised" License
79 stars 35 forks source link

Require MNE 1.0 or above #479

Open rob-luke opened 2 years ago

rob-luke commented 2 years ago

There are a number of places in the code where we have something like remove once MNE 1.0 is required. I would like to enforce that MNE 1.0+ is required for MNE-NIRS to reduce the required maintenance. What are your thoughts on this @larsoner? I know MNE-Python gives longer support, but these side packages don't have the same size development team.

larsoner commented 2 years ago

Most downstream mne- packages only support the latest release and main. So you could bump to 1.1 already here

rob-luke commented 2 years ago

Awesome, I will make a note about that in the docs and pull out that code as I see it.

rob-luke commented 1 year ago

Most downstream mne- packages only support the latest release and main

Great, when MNE-Python 1.4 is released lets add this constraint to the requirements and environment.yml files.

When is the next MNE release roughly scheduled @larsoner ? (I should also get back on discord, sorry)

larsoner commented 1 year ago

In the next week or two

rob-luke commented 1 year ago

Well, that's just worked out dandy then ;) Good time for some spring cleaning in MNE-NIRS too then.