MAINT: Tolerate new MNE behavior

[larsoner]

@hoechenberger should fix the failing test following mne-tools/mne-python#11212 where fiducials are estimated and added when implicitly converting to the head coordinate frame

[larsoner]

I also just pushed a commit to avoid using importlib_resources. It's causing a conda-forge build error:

conda-forge/mne-bids-feedstock#21

Keeping it would mean 1) adding to setup.py deps, probably dependent on Python version, 2) changing the import, probably with a try/except, 3) updating the feedstock. What I've implemented here instead is just data_dir = Path(__file__).parent / 'data' which requires none of these gymnastics, and hopefully is good enough.

[hoechenberger]

Sounds good to me. Some tests are still failing though!

[larsoner]

Those should be fixed by mne-tools/mne-python#11230

[codecov]

Codecov Report

Merging #1076 (f744663) into main (de529c4) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1076      +/-   ##
==========================================
- Coverage   95.21%   95.21%   -0.01%     
==========================================
  Files          24       24              
  Lines        3845     3844       -1     
==========================================
- Hits         3661     3660       -1     
  Misses        184      184              

Impacted Files	Coverage Δ
mne_bids/dig.py	98.26% <100.00%> (-0.01%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[larsoner]

Ready for review/merge from my end

[sappelhoff]

Thanks @larsoner

[sappelhoff]

we should probably have removed this line as well with this PR, right?

mne-bids/setup.cfg

Line 41 in ee27b43

importlib-resources

done in #1077

[larsoner]

Yes we should have, thanks @sappelhoff !