-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
API: mne.get_config #1112
Comments
+1 On Fri, Jan 31, 2014 at 6:20 PM, Denis A. Engemann <notifications@github.com
|
Should it return a json object? |
@saketkc simply a dictionary. Once you open a json file and represent it's contents in Python what you get is a dict. |
I think a |
Yes, my choice of |
@saketkc anyways, PR welcome :-) We can think about adding custom representations should we come to the conclusion that returning all json mappings as a dict is messy. E.g. as we did it with info objects https://github.com/mne-tools/mne-python/blob/master/mne/fiff/meas_info.py#L37 |
should be callable without arguments and should return all mappings
The text was updated successfully, but these errors were encountered: