Skip to content

Make mne/digitization private mne/_digitization#6697

Closed
massich wants to merge 6 commits intomne-tools:masterfrom
massich:make_digitization_private
Closed

Make mne/digitization private mne/_digitization#6697
massich wants to merge 6 commits intomne-tools:masterfrom
massich:make_digitization_private

Conversation

@massich
Copy link
Copy Markdown
Contributor

@massich massich commented Aug 22, 2019

Do not review. this goes after #6639

@massich massich force-pushed the make_digitization_private branch from f12fe3f to ab27c88 Compare August 22, 2019 11:27
@massich
Copy link
Copy Markdown
Contributor Author

massich commented Aug 22, 2019

Does anyone know why the CIs do not do what I expect??? they are not running.

@massich massich force-pushed the make_digitization_private branch from 30066de to c9cb583 Compare August 22, 2019 14:09
- Speed up morph map generation in :func:`mne.read_morph_map` by ~5-10x by using :func:`numba.jit` by `Eric Larson`_

- Add :class:`mne.digitization.Digitization` class to simplify montage by `Joan Massich`_
- Add ``Digitization`` class to simplify montage by `Joan Massich`_
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you now remove this entry?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

the whole thing? sure.

@agramfort
Copy link
Copy Markdown
Member

agramfort commented Aug 22, 2019 via email

@massich
Copy link
Copy Markdown
Contributor Author

massich commented Aug 22, 2019

closed in favor of #6700 (I've no idea why was not working. moving on!)

@massich massich closed this Aug 22, 2019
@massich massich deleted the make_digitization_private branch August 22, 2019 14:19
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.

2 participants