Back to the main page.

Bug 1404 - implement support for the neurosim spikes and signals file format

Status CLOSED FIXED
Reported 2012-04-02 17:20:00 +0200
Modified 2019-08-10 11:56:50 +0200
Product: FieldTrip
Component: fileio
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P4 enhancement
Assigned to: Bart Gips
URL:
Tags:
Depends on:
Blocks:
See also:

Robert Oostenveld - 2012-04-02 17:20:50 +0200

This follows up on a discussion I had with Jan van der Eerden (in person and by mail). The implementation has already been finished, this bug is a placeholder for the test script and follow-up discussions


Robert Oostenveld - 2012-04-02 17:31:03 +0200

I added a test script on the basis of the ascii files in /opt/neurosim/Examples/AlphaColumns/dense_smalltest roboos@mentat001> svn commit Adding test/test_bug1404.m Transmitting file data . Committed revision 5577.


Robert Oostenveld - 2012-04-11 16:48:39 +0200

I cleaned up my bugzilla list by changing the status from resolved (either fixed or wontfix) into closed. If you don't agree, please reopen the bug. Robert


Robert Oostenveld - 2012-12-12 09:44:56 +0100

Hi Bart, I suspect that your recent changes to neurosim have caused this older test script now to fail. Can you please have a look? See http://fieldtrip.fcdonders.nl/development/dashboard/r7143/test_bug1404 which reports the error unsupported header format (neurosim dir)


Bart Gips - 2012-12-12 10:24:06 +0100

(In reply to comment #3) Yes, the recent changes make it important that you specify which file to read the header from. (spikes, evolution or signals) In this case it should be spikes. However ft_read_header reads some information (e.g. sampling frequency) that was not written in the header on earlier versions of Neurosim. So I'll edit the test function and I'll ask Jan to update the examples.


Robert Oostenveld - 2012-12-12 10:32:58 +0100

(In reply to comment #4) Rather than updating the test, should the test not stay the same and should the FT code not be updated to allow for the continued use as in the test script? The test script might reflect the analysis script of an external user, and we don't want external users having to update their scripts whenever we make a change.


Bart Gips - 2012-12-12 10:37:57 +0100

(In reply to comment #4) Secondly, I realise that the dataset should not be 'spikes'. ft_preprocessing does not work on spike data, because spike data have a different structure (timestamps). (ft_read_spike vs ft_read_data) What do you want to preprocess here? membrane potentials?, LFPs? I assume LFP, I can implement something like this (LFPs are in the signals file as 'Extracellular V' for every simulated network) However, the LFP that neurosim calculates is still a little strange, so not very useful at the moment.


Bart Gips - 2012-12-12 10:40:32 +0100

(In reply to comment #5) I believe changing the test function is better, because if the user supplies a directory it is unclear what is meant. Different files in the neurosim directory have different headers and different sampling rates. Besides I believe, right now I am the only one using this (besides Jan and perhaps Eric Lowet). So it will be no problem to switch the input method now.


Robert Oostenveld - 2012-12-12 10:43:32 +0100

(In reply to comment #6) for some spike formats (but I don't know whether it applies to all) ft_preprocessing is able to read it and represent it as [00000010000100000101000100...]


Robert Oostenveld - 2012-12-12 10:44:34 +0100

(In reply to comment #7) let's discuss following the FT meeting. I believe there are still some design issues with the format that need to be resolved.


Bart Gips - 2012-12-12 10:50:58 +0100

(In reply to comment #9) We could implement that by letting ft_read_data perform ft_read_spike and then using ft_checkdata to convert to such a continuous data format. I'll be there this afternoon to discuss.


Bart Gips - 2012-12-18 09:37:14 +0100

neurosim_spikes can be read using (directly use the file as input): ft_read_spike (spike structure containing spike timepoints) ft_read_data (continuous data containing 0's and 1's) ft_read_header neurosim_signals can be read using: ft_read_data ft_read_header (both when using the file directly (='neurosim_signals') as input or when using a directory containing the 'signals' file (='neurosim_ds')) neurosim_evolution can be read using (directly use the file as input): ft_read_data ft_read_header


Robert Oostenveld - 2019-08-10 11:56:50 +0200

This closes a whole series of bugs that have been resolved (either FIXED/WONTFIX/INVALID) for quite some time. If you disagree, please file a new issue describing the issue on https://github.com/fieldtrip/fieldtrip/issues.