Skip to content

Conversation

@massich
Copy link
Contributor

@massich massich commented Apr 23, 2019

No description provided.

Copy link
Member

@larsoner larsoner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@larsoner larsoner changed the title watersheed MRG: watersheeeed Apr 23, 2019

.. versionadded:: 0.13.0
copy : bool
If True (default False), use copies instead of symlinks for surfaces
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

fnames = glob.glob(op.join(tempdir, '*proj.fif'))
assert len(fnames) == 1
fnames = glob.glob(op.join(tempdir, '*-eve.fif'))
assert len(fnames) == 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for files in ("*-eve.fif", "*proj.fif"):
    assert(len(glob.glob(op.join(tempdir, files))) == 1)

?

@jona-sassenhagen
Copy link
Contributor

Only stylistic comments

@larsoner larsoner merged commit e8ea87f into mne-tools:master Apr 23, 2019
@larsoner
Copy link
Member

Thanks @massich @jona-sassenhagen (tested locally after stylistic commit)

jeythekey pushed a commit to jeythekey/mne-python that referenced this pull request Apr 27, 2019
* Things unrelated to eeg_no_mri

* Update test_commands.py
@massich massich deleted the bem branch May 27, 2019 12:08
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.

3 participants