fixed bug w/ dcm2niix args

openneuropet · Sep 13, 2024 · d270d95 · d270d95
1 parent d901bc0
commit d270d95
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/matlab/dcm2niix4pet.m b/matlab/dcm2niix4pet.m
@@ -75,7 +75,7 @@ function dcm2niix4pet(FolderList,MetaList,varargin)
 dcm2niix_data.version = version(1);
 dcm2niix_data.returncode = 0;
 telemetry_data.dcm2niix = dcm2niix_data;
-telemetry_data.description = "Matlab_dcm2niix4pet.m"
+telemetry_data.description = "Matlab_dcm2niix4pet.m";
 
 if length(version) >= 1
     version_date = version{1}(6:end);
@@ -222,8 +222,6 @@ function dcm2niix4pet(FolderList,MetaList,varargin)
         outputdir = varargin{var+1};
     elseif strcmpi(varargin{var},'notrack')
         setenv('TELEMETRY_ENABLED', 'False')
-    else
-        error('unknown option %s',varargin{var})
     end
 
 end

diff --git a/matlab/telemetry.m b/matlab/telemetry.m
@@ -19,9 +19,12 @@ function telemetry(telemetry_data, input_path, output_path)
         end
 
         url = 'http://openneuropet.org/pet2bids/';
-        options = weboptions('MediaType', 'application/json');
-        response = webwrite(url, telemetry_data, options);
-
+        options = weboptions('MediaType', 'application/json', 'Timeout', 5);
+        try
+            response = webwrite(url, telemetry_data, options);
+        catch ME
+            % do nothing
+        end
     else
         % don't do anything
     end

diff --git a/scripts/matlab_conversions.m b/scripts/matlab_conversions.m
@@ -31,7 +31,7 @@
     [status, results] = system('which dcm2niix');
     if status == 0
        dcm2niix_path = results;
-        system([dcm2niix_path, ' -h'])
+       system(dcm2niix_path, ' -h')
     else
         print_me = ['This script expects to find dcm2niix at /opt/homebrew/bin, for a default brew installed dcm2niix.'];
         disp(print_me)