Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error getting units for 668755_2023-08-28 #127

Open
bjhardcastle opened this issue Sep 20, 2024 · 1 comment
Open

Error getting units for 668755_2023-08-28 #127

bjhardcastle opened this issue Sep 20, 2024 · 1 comment
Assignees

Comments

@bjhardcastle
Copy link
Member

Might be related to #126

Traceback (most recent call last):
  File "/opt/conda/lib/python3.9/site-packages/npc_ephys/units.py", line 269, in make_units_table_from_spike_interface_ks25
    _ = future.result()
  File "/opt/conda/lib/python3.9/concurrent/futures/_base.py", line 439, in result
    return self.__get_result()
  File "/opt/conda/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result
    raise self._exception
  File "/opt/conda/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/opt/conda/lib/python3.9/site-packages/npc_ephys/units.py", line 177, in _device_helper
    awc = get_amplitudes_waveforms_channels_ks25(
  File "/opt/conda/lib/python3.9/site-packages/npc_ephys/units.py", line 77, in get_amplitudes_waveforms_channels_ks25
    _templates_mean = spike_interface_data.get_nwb_units_device_property(
  File "/opt/conda/lib/python3.9/site-packages/npc_ephys/spikeinterface.py", line 363, in get_nwb_units_device_property
    self.device_indices_in_nwb_units(probe)
  File "/opt/conda/lib/python3.9/site-packages/npc_ephys/spikeinterface.py", line 353, in device_indices_in_nwb_units
    [
  File "/opt/conda/lib/python3.9/site-packages/npc_ephys/spikeinterface.py", line 354, in <listcomp>
    npc_session.ProbeRecord(device)
  File "/opt/conda/lib/python3.9/site-packages/npc_session/records.py", line 135, in __new__
    return super().__new__(cls, cls.parse_id(content))
  File "/opt/conda/lib/python3.9/site-packages/npc_session/records.py", line 213, in parse_id
    raise ValueError(
ValueError: ProbeRecord requires a string containing `probe` followed by a letter A-F, or a single capital A-F alone, not 'Neuropixels 1.0'
@bjhardcastle bjhardcastle added this to the paper1-datacube milestone Sep 20, 2024
@bjhardcastle bjhardcastle self-assigned this Sep 20, 2024
@bjhardcastle
Copy link
Member Author

bjhardcastle commented Sep 20, 2024

This seems to be a badly-named device in the NWB from the sorting pipeline.. must be fixed now as no other sessions had this.

Re-sorting...

Edit:
This was caused by the upload pipeline, not sorting: spikeinterface gave bad device names. Will be fixed, then re-sorted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant