Coder Social home page Coder Social logo

Comments (4)

djoshea avatar djoshea commented on July 21, 2024

Hi Bill, thanks! It's been a while since I've looked at this code, and at the time (2018) I was guessing at the metadata, and before I knew about your CatGT and TPrime.

Can you unpack what you mean by misspell here? What were you doing to generate a new metadata file? Were you running a processing pipeline like this https://djoshea.github.io/neuropixel-utils/imec_dataset/#modifying-a-dataset-during-copy-to-new-location

I think in the call to writeModifiedAPMeta, these are the three fields that I prefix the fields with a tilde because that's how they appear in my .{band}.meta files, and stripping the tilde led to errors with other tools.

function writeModifiedAPMeta(imec, varargin)

What are the fields supposed to be? What is the correct output? Thanks!

from neuropixel-utils.

billkarsh avatar billkarsh commented on July 21, 2024

The tilde is required. I am reacting to a bug report about a CatGT crash that I got from a user. The cause was metadata created by neuropixel-utils where the tilde character was absent. I've now looked at your code and it does appear to explicitly add a tilde for those items, though I have never run/tested your code myself. I am guessing that the user's version predates this. If the tilde is present in current code you can close the issue. Thanks for checking this out.

from neuropixel-utils.

djoshea avatar djoshea commented on July 21, 2024

Ah, okay. Looks like I fixed this in 2021 ae56e18

Please send the user my way if it's not fixed by a pull and I can help track down the issue.

Wish I'd known of the full complement of tools you'd written when I started writing my toolkit! Thanks!

from neuropixel-utils.

zhangxiaoxing avatar zhangxiaoxing commented on July 21, 2024

I am that user, and I'm just about to dig into things when you guys already resolved it. That's impressive. Thanks everyone!

Just in case someone else searched for this, adding 3 tildes (~) before the imroTbl, snsChanMap, snsShankMap entry in the modified metafile is sufficient to restore CatGT functionality.

from neuropixel-utils.

Related Issues (13)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.