search

czarrar / connectir

Fast and memory efficient approach for connectome-wide association studies in R and C++. Uses custom implementation of multivariate distance matrix regression to associate brain and behavior.
15 stars 9 forks source link

Fix Error non-numeric argument to binary operator #5

Closed neerajt closed 8 years ago

neerajt commented 8 years ago

Traced a problem to memlimit.R:19 when using ROI text files; inputs were being read as non-numeric. This patch coerces inputs to numeric for the get_subdist_memlimit function for calculating memory requirements.

Full output with error:

connectir_subdist.R -i functional_path_list.txt --in2D2 --ztransform --bg ~/Downloads/connectir/data/standard_4mm.nii.gz --memlimit 4 --overwrite test_output

Loading niftir version < 1.0.

Setting 1 parallel forks
Setting 1 threads for matrix algebra operations
...using Intel's MKL
Checking required inputs
Checking optional inputs
...NOT using 2nd set of functional images
Preparing functional information

Preparing functionals
Preparing functional info
Preparing functional information

Computing brain mask
final mask has 100.00% non-zero voxels
Memory?
Determining memory demands

An error was detected: 
non-numeric argument to binary operator 

Called by: 
x * nvoxs

Saving options...

Removing everything from memory
...sucesss