-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MRG] Simplify raw_to_bids #106
Changes from all commits
6dbf9e2
9f1afb2
e56fbc5
2329d6f
e12fc43
3893742
8c6ec21
bb8b955
9aebb31
b56b1c9
5a445c5
208b815
c7e2ae4
afab4e2
cc14f9c
ef9e116
b402150
e1386b8
2eaccda
1cf4e53
47f8e62
976aba0
5afa95f
2074a66
75b02c9
003a811
cc5ad78
2444990
0ea50dd
450310b
2989e65
a471299
30df0b4
21c6052
665fbf5
6e5beea
4c378c5
75b0cb6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,7 +13,9 @@ MNE BIDS (:py:mod:`mne_bids.mne_bids`): | |
.. autosummary:: | ||
:toctree: generated/ | ||
|
||
raw_to_bids | ||
write_raw_bids | ||
make_bids_basename | ||
make_bids_folders | ||
|
||
Utils (:py:mod:`mne_bids.utils`): | ||
|
||
|
@@ -23,10 +25,9 @@ Utils (:py:mod:`mne_bids.utils`): | |
:toctree: generated/ | ||
|
||
print_dir_tree | ||
make_bids_folders | ||
make_bids_filename | ||
make_dataset_description | ||
copyfile_brainvision | ||
copyfile_eeglab | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do you need to make these functions public? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. at least the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I guess these are useful functions? I don't know ... There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think they fit on the api page, but I think they should remain public through the utils There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. But we have an example using these functions. Do you propose to remove the example too? |
||
|
||
Datasets (:py:mod:`mne_bids.datasets`): | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we can note the mne-python current version as the minimal in the text above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure what you mean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we don't specify anywhere minimal version number. like, we should have a section that says mne >= 0.17.X
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's probably because we don't know them. I know only for
MNE
we need >=0.17.x. For the rest I would say the "latest" :-)