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

[Feature]: EDF with multiple offsets #1106

Open
2 tasks done
bendichter opened this issue Oct 3, 2024 · 1 comment
Open
2 tasks done

[Feature]: EDF with multiple offsets #1106

bendichter opened this issue Oct 3, 2024 · 1 comment

Comments

@bendichter
Copy link
Contributor

What would you like to see added to NeuroConv?

I was just in a meeting with Hezekiah Branch, who wants to convert EDF files to NWB and share them on DANDI. When following the docs from the conversion gallery for EDF, we got an error that we can't handle data that has different channel offsets. apparently this file specifies different channel offsets for different channels. I know we've seen different conversion factors, but I can't recall different offsets. Indeed, looking at the docs for ElectricalSeries in PyNWB, it looks like we allow for multiple conversion factors but not multiple offsets. Have we dealt with this kind of thing before? Maybe there's a way around this. I have asked Hezekiah to share one of these files with us so we can dig in and see what is going on.

Is your feature request related to a problem?

No response

Do you have any interest in helping implement the feature?

Yes.

Code of Conduct

@h-mayorquin
Copy link
Collaborator

h-mayorquin commented Oct 8, 2024

Did they share some data that we can use to work on this?

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

No branches or pull requests

2 participants