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

API: Change default to _cache #778

Closed larsoner closed 1 year ago

larsoner commented 1 year ago

Before merging …

Closes #723

@hoechenberger feel free to mark as merge-when-green if you're happy.

hoechenberger commented 1 year ago

@larsoner Any particular reason not to call it .cache?

larsoner commented 1 year ago

@larsoner Any particular reason not to call it .cache?

From the discussion in the linked issue https://github.com/mne-tools/mne-bids-pipeline/issues/723#issuecomment-1477788187 it seemed like .cache was no good / unsafe but _cache was okay