Add support for Biocam brw v4.x files

[mahlzahn]

• Missing: test file for brw v4.x file format

See also NeuralEnsemble/python-neo/issues/1324 and corresponding NeuralEnsemble/python-neo/pull/1326

[alejoe91]

Thanks @mahlzahn

Could you enable the option "Allow edits by maintainers", so that the code is automatically re-formatted?

[codecov-commenter]

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (949da47) 88.32% compared to head (368d3f6) 88.41%.
Report is 98 commits behind head on main.

Files	Patch %	Lines
src/probeinterface/io.py	61.53%	5 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #216      +/-   ##
==========================================
+ Coverage   88.32%   88.41%   +0.09%     
==========================================
  Files          10       10              
  Lines        1644     1718      +74     
==========================================
+ Hits         1452     1519      +67     
- Misses        192      199       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mahlzahn]

Thanks @mahlzahn

Could you enable the option "Allow edits by maintainers", so that the code is automatically re-formatted?

I already did this, but only after submission. Once I have the test file and tests implemented, I guess the automatic re-formatting should run on the new commit.

[alejoe91]

Sounds good! Thanks!

[mahlzahn]

@alejoe91 I don’t know if you prefer that the new test file is added also here. What do you think?

[alejoe91]

@mahlzahn since the data is small, I think it's ok to add it here. @samuelgarcia ?

[alejoe91]

@mahlzahn sorry for the enormous delay.

I have a couple of small suggestions and comments!

[alejoe91]

@mahlzahn I implemented the couple of changes I suggested. It's ok to merge on my side!

[mahlzahn]

Thanks, @alejoe91!

@mahlzahn I implemented the couple of changes I suggested. It's ok to merge on my side!