NeurodataWithoutBorders / nwb-guide

NWB GUIDE is a desktop app that provides a no-code user interface for converting neurophysiology data to NWB.
https://nwb-guide.readthedocs.io/
MIT License
22 stars 5 forks source link

"Quality is a required field" when uploading Kilosort4 #869

Open oliriley opened 2 months ago

oliriley commented 2 months ago

Describe the issue

When attempting to convert a paired Open Ephys recording with its associated spike sorting, I encountered an error stating "quality" was required in the table describing the units: Screenshot 2024-06-21 120318

No quality.npy or similar is present in the kilosort results folder. The data were processed with kilosort 4 and the same issue arrises whether uploading a Kilosort session alone or a Phy session after curation.

Steps to Reproduce

  1. Begin new conversion
  2. Select Kilosort or Phy interface
  3. Provide path to sorting results folder containing params.py
  4. Hit next to extract metadata
  5. Error under "Ecephys" -> "Summarized Units"

Operating System

Windows 11 Home

GUIDE Version

1.0.1 (I think)

Code of Conduct

Yes

Did you confirm this issue was not already reported?

Yes

CodyCBakerPhD commented 2 months ago

Hi @oliriley

Unfortunately, I'm having trouble reproducing this on our current example data (for Kilosort/Phy, that is) - would you be able to share your data over Google Drive or Dropbox so I can try to get to the bottom of this? (spike sorting data may also be small enough to zip and send over GitHub)

Two other things to try might be

i) Use the Phy interface instead (KS3 as I recall is essential equivalent to Phy minus certain unit classifications) I see you tried that and the problem persists ii) Go ahead and write in a value for that field on the table; 'unknown' is also acceptable as a placeholder Nevermind, I don't think we have enabled interactive editing of those columns on the summary table

CodyCBakerPhD commented 2 months ago

Another option that I believe should work is

iii) delete the row with the name 'quality' on the UnitsProperties tab and that column of the Summarized units should disappear

CodyCBakerPhD commented 2 months ago

But I see that (iii) is also blocked by #873