#38 Extend openephys files to "binary" default format

Avoinna
3 vuotta sitten avasi alejoe91 · 10 kommenttia

Hi guys,

I have a smallish Open Ephys binary file of a Neuropixels recording (I think @sprenger has the same).

Can I make a PR to add it or is it too large?

Cheers Alessio

Hi guys, I have a smallish Open Ephys binary file of a Neuropixels recording (I think @sprenger has the same). Can I make a PR to add it or is it too large? Cheers Alessio
sprenger kommentoitu 3 vuotta sitten
Omistaja

Hi @Alejoe91,

if we are talking about the same dataset, then one of the two continuous data files still has a size of 23MB. Do you think it would be possible to replace this e.g. by a duplication of the other (smaller) file or shorten this file a bit?

@apdavison & @samuelgarcia: Do you agree to #37 or how should we proceed from now on with PRs for this repository?

Hi @Alejoe91, if we are talking about the same dataset, then one of the two continuous data files still has a size of 23MB. Do you think it would be possible to replace this e.g. by a duplication of the other (smaller) file or shorten this file a bit? @apdavison & @samuelgarcia: Do you agree to #37 or how should we proceed from now on with PRs for this repository?
Alessio Buccino kommentoitu 3 vuotta sitten
Yhteistyökumppani

Hi Julia,

I would have to ask Josh Siegle for another shorter dataset..this is already 1s.

Can we make an exception to the 10Mb rule given the git-annex update?

Let me know. I can then amke a PR as suggested in https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/37

Hi Julia, I would have to ask Josh Siegle for another shorter dataset..this is already 1s. Can we make an exception to the 10Mb rule given the `git-annex` update? Let me know. I can then amke a PR as suggested in https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/37
Samuel Garcia kommentoitu 3 vuotta sitten
Omistaja

Hi Alessio. If you create this, could you name the folder "openpehysbinary" to mtach the new class "OpenEphysBinaryIO" ?

Sam

Hi Alessio. If you create this, could you name the folder "openpehysbinary" to mtach the new class "OpenEphysBinaryIO" ? Sam
sprenger kommentoitu 3 vuotta sitten
Omistaja
@samuelgarcia: This should be closed via https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/39 ,right?
sprenger kommentoitu 3 vuotta sitten
Omistaja

Hi @Alejoe91, I would be also interested in test examples dataset with spikes and waveforms. It seems the video tracking dataset has spike files, but there are no spike times or waveforms contained. Do you have access to a dataset that has these features and can provide it?

Hi @Alejoe91, I would be also interested in test examples dataset with spikes and waveforms. It seems the video tracking dataset has spike files, but there are no spike times or waveforms contained. Do you have access to a dataset that has these features and can provide it?
Alessio Buccino kommentoitu 3 vuotta sitten
Yhteistyökumppani

Hi @sprenger

I can ask for a test example with spike sorted data. I'm not sure waveforms are saved in the binary OE format. We can check that!

Hi @sprenger I can ask for a test example with spike sorted data. I'm not sure waveforms are saved in the binary OE format. We can check that!
sprenger kommentoitu 3 vuotta sitten
Omistaja

@Alejoe91 Thanks! In the format specifications a spike_waveforms.npy is listed (https://open-ephys.github.io/gui-docs/User-Manual/Recording-data/Binary-format.html) This would be interesting to include.

@Alejoe91 Thanks! In the format specifications a spike_waveforms.npy is listed (https://open-ephys.github.io/gui-docs/User-Manual/Recording-data/Binary-format.html) This would be interesting to include.
Alessio Buccino kommentoitu 3 vuotta sitten
Yhteistyökumppani

@sprenger I do have a test file. Should I send it to you via email?

@sprenger I do have a test file. Should I send it to you via email?
sprenger kommentoitu 3 vuotta sitten
Omistaja

@Alejoe91 If it's not directly suited for adding in this repository, then yes please. Alternatively, I can also add it later after testing it first. Thanks!

@Alejoe91 If it's not directly suited for adding in this repository, then yes please. Alternatively, I can also add it later after testing it first. Thanks!
Alessio Buccino kommentoitu 3 vuotta sitten
Yhteistyökumppani

Yes I think it can be added. Not sure it has more than one unit, but it has a spike group with spike_waverofms.npy

Yes I think it can be added. Not sure it has more than one unit, but it has a spike group with spike_waverofms.npy
Kirjaudu sisään osallistuaksesi tähän keskusteluun.
Ei tunnistetta
Ei merkkipaalua
Ei osoitettua
3 osallistujaa
Ladataan...
Peruuta
Tallenna
Sisältöä ei vielä ole.