Skip to content

Conversation

@larsoner
Copy link
Member

Closes #7744

@rob-luke this code only takes 1 sec now, no idea how long it takes on master because I lost patience waiting after 30 seconds or so:

import time
import mne
fnirs_data_folder = mne.datasets.fnirs_motor.data_path()
fnirs_raw_dir = fnirs_data_folder + '/Participant-1'
raw = mne.io.read_raw_nirx(fnirs_raw_dir)
t0 = time.time()
raw.save('/dev/null', verbose='error')
print(time.time() - t0)

@kylemath
Copy link
Contributor

This change is helpful for us and we will add to our import function too (our data files for different blocks sometimes have different sized headers)

@rob-luke
Copy link
Member

Thanks @larsoner, I appreciate you fixing this.

@larsoner
Copy link
Member Author

Okay, in it goes, thanks for looking @kylemath @rob-luke . In the future feel free to click the green "Review changes" in the upper right of the files-changed page and "Approve" or add some comments and "Request changes" etc., too, if you want. It's always good to get extra eyes (and know that there have been extra eyes) on any code changes...

@larsoner larsoner merged commit 861e825 into mne-tools:master May 12, 2020
@larsoner larsoner deleted the nirx branch May 12, 2020 14:33
@rob-luke
Copy link
Member

feel free to click the green "Review changes" in the upper right

Thanks, will do in the future.

larsoner added a commit to larsoner/mne-python that referenced this pull request May 19, 2020
* upstream/master: (74 commits)
  FIX: Correct a bug in find_bads_eog (mne-tools#7797)
  [MRG] split_naming='bids' changes from _part-%d to _split-%d (mne-tools#7794)
  MRG, MAINT, DOC: Remove spyder (mne-tools#7796)
  MAINT: fixes for linkcheck (mne-tools#7762)
  [WIP] Update ieeg data example for ECoG (mne-tools#7768)
  fix examples/tutorials [circle full] (mne-tools#7786)
  MAINT: Clean up VTK and add to pre on Azure (mne-tools#7780)
  ENH: Add matplotlib animation support [skip travis] (mne-tools#7783)
  MRG, API: change out_type default in permutation_cluster_(1samp_)test (mne-tools#7781)
  DOC: docstring fixes (mne-tools#7777)
  MRG, ENH: Add tol_kind option (mne-tools#7736)
  MRG, DOC: Notes about info (mne-tools#7772)
  ENH: Speed up NIRx read without preload (mne-tools#7759)
  Minor plot_raw aes improvement (mne-tools#7770)
  MRG, FIX: Fixes for BEM contours (mne-tools#7763)
  MRG, STY: Fix E741 (mne-tools#7767)
  MRG, ENH - Plot optodes in plot_alignment for fNIRS channels (mne-tools#7747)
  FIX: Update NIH support [skip travis] (mne-tools#7766)
  MAINT: Bump tol for gamma map test (mne-tools#7764)
  MRG, FIX: Fix MRI orientations (mne-tools#7725)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Writing NIRS files to FIF is surprisingly slow

3 participants