nwb-guide icon indicating copy to clipboard operation
nwb-guide copied to clipboard

Issue with NWB .plx sorted data

Open ajcheng1 opened this issue 1 year ago • 8 comments

Describe the issue

I want to convert several .plx (plexon) files into .nwb, but I am getting this error with one of the files I generated myself and one of the example files plexon had online (https://plexon.com/software-downloads/#software-downloads-data-samples | see "Sample PLX File – Four Channel Demo")

After I fill out the file metadata and hit next, I get this error "[sub-test2/ses-test2]: Form is not allowed to have the additional property "UnitColumns"." image

Steps to Reproduce

  1. Go to NWB Guide (GUI)
  2. fill in entries
  3. add plexon Sorting (not plexon recording) .plx files
  4. add example .plx file linked above
  5. fill out metadata
  6. hit next to see the error

Operating System

Windows 11

GUIDE Version

1.0.2

Code of Conduct

Yes

Did you confirm this issue was not already reported?

Yes

ajcheng1 avatar Aug 12 '24 17:08 ajcheng1

Was Plexon the only interface selected in your pipeline?

This is likely related to #873

CodyCBakerPhD avatar Aug 12 '24 17:08 CodyCBakerPhD

yes, I only added plexon sorting. I don't see unit columns image

ajcheng1 avatar Aug 12 '24 17:08 ajcheng1

Oh, thank you for specifying PlexonSorting (I had assumed PlexonRecording)

This is indeed related to #873 then, in particular the case of an empty table (which thus can't be edited to remove from metadata)

This special case might be a bit easier to solve via a backend cleanup prior to sending to the web form

CodyCBakerPhD avatar Aug 12 '24 18:08 CodyCBakerPhD

ok I see. Thanks for the heads ups. I try using the neuroconv python program using the example plexon data provided in the link (File_plexon_3.plx) but there seem to be some hurdles with that. https://github.com/catalystneuro/neuroconv/pull/206 I can add a comment to that github repository with the issue I am encountering.

ajcheng1 avatar Aug 12 '24 18:08 ajcheng1

I try using the neuroconv python program using the example plexon data provided in the link

An excellent next step!

I can add a comment to that github repository with the issue I am encountering.

Please do; raise an issue over there with your code and the error+traceback you encounter

CodyCBakerPhD avatar Aug 12 '24 19:08 CodyCBakerPhD

I added support for this in the latest version of neuroconv.

https://github.com/catalystneuro/neuroconv/pull/1013

However, after discussing this in Spikeinterface it surfaced that the user's data plexon file had only waveforms and that was the reason it could not be opened with Spikeinterface (and therefore neuroconv and the guide):

https://github.com/SpikeInterface/spikeinterface/issues/3343

h-mayorquin avatar Sep 18 '24 02:09 h-mayorquin

I suggest we close this issue unless @ajcheng1 has something else to add.

h-mayorquin avatar Sep 18 '24 02:09 h-mayorquin

Apologies, I was on leave for a bit. Let me see if there is a way to extract Axion data into plexon without losing any data. I'll get back to you.

ajcheng1 avatar Sep 23 '24 18:09 ajcheng1