Implement Biocam reader for 3Brain system

[alejoe91]

• test files
• tests
• info about gain and offset

[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/biocamrawio.py:

Line 32:73: W291 trailing whitespace
Line 121:100: E501 line too long (110 > 99 characters)
Line 168:100: E501 line too long (106 > 99 characters)

• In the file neo/test/rawiotest/test_biocam.py:

Line 16:9: E123 closing bracket does not match indentation of opening bracket's line

Comment last updated at 2022-01-25 11:57:10 UTC

[samuelgarcia]

@alejoe91 : any news here ?
Do you think it will be ready soon or do I move this to 0.11 release ?

[alejoe91]

@alejoe91 : any news here ?
Do you think it will be ready soon or do I move this to 0.11 release ?

Not this week on my side. So you can move it to next release if you are releasing this week!

[JuliaSprenger]

Hi @alejoe91 Is this PR still alive?

[alejoe91]

Hi @alejoe91 Is this PR still alive?

Hi @JuliaSprenger, let's keep it open. We got stuck because it's unclear how the conversion factors are stored in the file. Or do you prefer to close it and re-open when we get time to work on it?

[JuliaSprenger]

Ok, then we keep it open and I ping again in some weeks.

[alejoe91]

Tests depend on this GIN PR: https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/66

[alejoe91]

@JuliaSprenger @samuelgarcia this is ready to review. Also added a test file on GIN (PR still need revision)

[alejoe91]

@JuliaSprenger this seems to be failing for a SpikeGLX problem, but the biocam test passes

[samuelgarcia]

Hi Alessio.
Cool. Thanks.
I will have a look next week.

[samuelgarcia]

@alejoe91 : can you rebase on master and so re run tests ?

[alejoe91]

Sure