Fix bug with too many JSON files found during empty room matching

mne-tools / mne-bids-pipeline

Automatically process entire electrophysiological datasets using MNE-Python.

https://mne.tools/mne-bids-pipeline/

BSD 3-Clause "New" or "Revised" License

140 stars 67 forks source link

Fix bug with too many JSON files found during empty room matching #743

Closed allermat closed 1 year ago

allermat commented 1 year ago

…steps/init/_02_find_empty_room.py

Before merging …

[ ] Changelog has been updated (docs/source/changes.md)

Closes #717

welcome[bot] commented 1 year ago

Hello! 👋 Thanks for opening your first pull request here! ❤️ We will try to get back to you soon. 🚴🏽‍♂️

larsoner commented 1 year ago

Just pushed a commit to update the changelog and marked for merge when green, thanks in advance for the fix @allermat !

(Ideally we would have some test case/dataset that had this problem, but I don't think that's so easy -- so good to merge enough with no explicit test!)

welcome[bot] commented 1 year ago

🎉 Congrats on merging your first pull request! 🥳 Looking forward to seeing more from you in the future! 💪