trendscenter / coinstac

Collaborative Informatics and Neuroimaging Suite Toolkit for Anonymous Computation
MIT License
45 stars 19 forks source link

Correct nifti test data #1541

Open rssk opened 1 year ago

rssk commented 1 year ago

Task Description

The test data in algorithm-dev is wrong here https://github.com/trendscenter/coinstac/tree/master/algorithm-development/test-data/nifti-test-data

replace it with https://github.com/trendscenter/coinstac-regression-vbm/tree/master/test/input from the 4 local directories here

Nonzzo commented 1 year ago

Looks like moving files from one repo to another while retaining git history. am I right?

Nonzzo commented 1 year ago

I have made a pull request that should meet the requirement. pls check

hvgazula commented 1 year ago

Regarding https://github.com/trendscenter/coinstac-regression-vbm/tree/master/test/input, it looks like files from different datasets are mixed here. Please consider creating a test suite to keep the data and corresponding input specs separate to facilitate ease of testing. https://github.com/trendscenter/coinstac/discussions/1564