spikeglx naming debug #1240

[samuelgarcia]

#1240

[samuelgarcia]

@DradeAW can you test this ?

[DradeAW]

I now have a different error:

recording = se.SpikeGLXRecordingExtractor(".", stream_name="imec0.ap")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/export/home1/users/nsr/wyngaard/dev/spikeinterface/spikeinterface/spikeinterface/extractors/neoextractors/spikeglx.py", line 67, in __init__
    probe = pi.read_spikeglx(meta_filename)
  File "/export/home1/users/nsr/wyngaard/dev/spikeinterface/probeinterface/probeinterface/io.py", line 1119, in read_spikeglx
    meta = parse_spikeglx_meta(meta_file)
  File "/export/home1/users/nsr/wyngaard/dev/spikeinterface/probeinterface/probeinterface/io.py", line 1149, in parse_spikeglx_meta
    key, val = line.split('=')
ValueError: too many values to unpack (expected 2)

I'm guessing this is because of the 3rd line in the meta file containing multiple = signs?

[samuelgarcia]

This is in probeinterface you need to update probeinterface I think

[DradeAW]

Same problem, just the line numbers that are different:

recording = se.SpikeGLXRecordingExtractor(".", stream_name="imec0.ap")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/export/home1/users/nsr/wyngaard/dev/spikeinterface/spikeinterface/spikeinterface/extractors/neoextractors/spikeglx.py", line 67, in __init__
    probe = pi.read_spikeglx(meta_filename)
  File "/export/home1/users/nsr/wyngaard/dev/spikeinterface/probeinterface/probeinterface/io.py", line 1133, in read_spikeglx
    meta = parse_spikeglx_meta(meta_file)
  File "/export/home1/users/nsr/wyngaard/dev/spikeinterface/probeinterface/probeinterface/io.py", line 1163, in parse_spikeglx_meta
    key, val = line.split('=')
ValueError: too many values to unpack (expected 2)

[samuelgarcia]

Does the neo.SpikeGLXIO working ?

[DradeAW]

Yes, both neo.rawio.SpikeGLXRawIO().parse_header() and neo.io.SpikeGLXIO seem to be working :)

[samuelgarcia]

So we need to make a equivalent patch in probeinterface!

[alejoe91]

This broke both spikeinterface and neuroconv latest tests!

• spikeinterface: https://github.com/alejoe91/spikeinterface/actions/runs/4607592885/jobs/8142306792
• neuroconv: https://github.com/catalystneuro/neuroconv/actions/runs/4606909733/jobs/8140786911?pr=395

[alejoe91]

I think it messes up with the nidq stream name...

[JuliaSprenger]

@samuelgarcia Do you mind looking into fixing also this?