pandas about ephypype HOT 6 OPEN

format_electrodes_xls is just a specific example from an Excel file, maybe it is better to be removed. The version ImportBrainVisaAscii could be removed as well, but if we want the package to handle multiples modalities, it may be good to have as much import format as possible, thus implying multiple packages. Having said that, this particular fuction requires the user to use export ascii frile from BrainVision, there are other ways to handle BrainVision data, possibly with numpy but evn better would be with neo (hence another optional package). What about make a test if the package is installed to allow the use of particular optional function?

from ephypype.

+1 to remove any code that is not used.

@annapasca @davidmeunier79 you have to explain to me why you need different nodes for different files. In MNE, you should be able to import brainvision to raw object. Why do you need another class/function for that?

from ephypype.

@jasmainak unfortunately life is not that simple. BrainVision is full of bugs, and when I had to interfere with it, and the export of BrainVision would be quite different depending on what you would have to export (type of EEG signal and intraEEG for example) and on what kind of analysis (windows segmenting, artefact removal, etc.). So I am not sure what you mean by "raw object", but it appeared to be easier in many cases to export it as ASCII. Having said that it has been a while since I touched it ...

from ephypype.

this is an MNE raw object: https://martinos.org/mne/dev/generated/mne.io.Raw.html. It's basically how you represent the multivariate time series in MNE along with the metadata.

Here is the reader for brainvision in MNE: https://www.martinos.org/mne/stable/generated/mne.io.read_raw_brainvision.html.

If there are bugs there, they should be fixed via pull requests instead of reinventing the wheel ...

from ephypype.

we are talking about a piece of code that was written 2 years ago:
https://github.com/davidmeunier79/neuropype_ephy/commits/master/neuropype_ephy/nodes/import_data.py
I am not even sure the read_raw_brainvision of MNE-python was developed at the time (see history).
if you feel like rewriting these functions in the new framework, you are welcome

from ephypype.

The brainvision reader is from 2013. See here: mne-tools/mne-python#878.

from ephypype.