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

Issue with NWB .plx sorted data #909

Open
ajcheng1 opened this issue Aug 12, 2024 · 8 comments
Open

Issue with NWB .plx sorted data #909

ajcheng1 opened this issue Aug 12, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@ajcheng1
Copy link

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

@CodyCBakerPhD
Copy link
Collaborator

Was Plexon the only interface selected in your pipeline?

This is likely related to #873

@ajcheng1
Copy link
Author

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

@CodyCBakerPhD
Copy link
Collaborator

CodyCBakerPhD commented Aug 12, 2024

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

@ajcheng1
Copy link
Author

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.

@CodyCBakerPhD
Copy link
Collaborator

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 CodyCBakerPhD added the bug Something isn't working label Aug 13, 2024
@h-mayorquin
Copy link

h-mayorquin commented Sep 18, 2024

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

catalystneuro/neuroconv#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):

SpikeInterface/spikeinterface#3343

@h-mayorquin
Copy link

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

@ajcheng1
Copy link
Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants