Back to the main page.

Bug 2215 - do not store test data in the svn/git repository

Status CLOSED FIXED
Reported 2013-07-01 10:56:00 +0200
Modified 2013-09-18 13:19:14 +0200
Product: FieldTrip
Component: core
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P3 normal
Assigned to: Robert Oostenveld
URL:
Tags:
Depends on:
Blocks:
See also:

Robert Oostenveld - 2013-07-01 10:56:01 +0200

On 28 Jun 2013, at 14:32, Herring, J.D. (Jim) wrote: As of June 13th Github has imposed a file size limit of 100mb (See: https://help.github.com/articles/working-with-large-files). I wasn't able to push my local copy to my forked online fieldtrip distribution after pulling from the 'blessed' fieldtrip distribution because of a 223mb mat-file in fieldtrip/test (test_bug1984_2187.mat). I thought you might be interested in this as this may cause problems with synchronizing between SVN and Github. I think this problem only relates to .mat files in fieldtrip/test we should consider moving all test data to home/common if we want to continue using Github (and if this causes problems for us, of course).


Robert Oostenveld - 2013-07-01 10:56:42 +0200

sorry Daniel, I accidentally added you to the CC of this bug. I removed you again.


Robert Oostenveld - 2013-07-01 11:00:50 +0200

Note to all: we should not add test data to fieldtrip/test, but store it in /home/common/matlab/fieldtrip/data/test. The consequence of having it in the repository (besides the one mentioned by Jim) is that it is added to each and every zip file that people download from the ftp server. We don't want to distribute the test data and we don't want the fieldtrip release to be unnecessary big. To give you an idea mbp> du -hs test/ 668M test/ there is presently 668 MB of stuff in the test directory. The fieldtrip release version is a 200MB zip file.


Robert Oostenveld - 2013-07-01 11:39:53 +0200

I went through a lot of the test files and changed them. See http://code.google.com/p/fieldtrip/source/detail?r=8291 for details or alternatively on https://github.com/fieldtrip/fieldtrip/commit/e8f7d9bcd0adfba766854fb4c5e3bf89bc950e52


Robert Oostenveld - 2013-07-01 11:42:04 +0200

the only one remaining is test_SPM_ft_integration.mat, which should remain in the test directory so that Vladimir can maintain it.


Robert Oostenveld - 2013-07-01 11:52:30 +0200

It helps quite a bit in the repository mbp> du -hs test/ 28M test/ But it won't make a difference for the release version, I just downloaded it and saw that the test directory is not in the zip.


Robert Oostenveld - 2013-09-18 13:19:14 +0200

closed multiple bugs that have been resolved