Openephys check consistency experiments

[alejoe91]

This PR improves the checks the OpenEphysBinaryRawIO by:

1. pre-mapping folder structure in terms of recording nodes, experiments, and recordings
2. assigning block_index and seg_index after pre-mappping (it is possible that some experiments/recordings are deleted by the user because of errors etc.)
3. adds checks to ensure that:
   • experiments (blocks) are consistent acrosse multiple nodes (if present)
   • streams are consistent across recordings (segments)
   • the same streams are present in all experiments (blocks)
     In case the latter check fails, there is a new option to use the argument experiment_names to select a subset of experiments with consistent streams to be loaded.

[pep8speaks]

Hello @alejoe91! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file neo/rawio/openephysbinaryrawio.py:

Line 335:69: W291 trailing whitespace
Line 424:65: E261 at least two spaces before inline comment
Line 429:100: E501 line too long (104 > 99 characters)
Line 433:100: E501 line too long (120 > 99 characters)
Line 457:65: E261 at least two spaces before inline comment
Line 540:100: E501 line too long (101 > 99 characters)

Comment last updated at 2022-08-31 09:06:25 UTC

[alejoe91]

@samuelgarcia i don't know why core tests failed! I only change one rawio file... can you check and in case retrigger?

[samuelgarcia]

Thanks Alessio.
This is really cool.

For me this OK to be merge.

@JuliaSprenger : I don't known if you want to have a look.
We need it for the release.

[samuelgarcia]

Test fails from something unrelated (neurlynx) I re run the test to check.
Unfortunatly the entire dataset will be download again (because test did not finish...)