Merge branch 'master' into remi-eyetracker

Author: Remi-Gau

.all-contributorsrc

@@ -13,7 +13,10 @@
       "contributions": [
         "code",
         "design",
-        "doc"
+        "doc",
+        "userTesting",
+        "ideas",
+        "bug"
       ]
     },
     {
@@ -24,7 +27,10 @@
       "contributions": [
         "code",
         "design",
-        "doc"
+        "doc",
+        "userTesting",
+        "ideas",
+        "bug"
       ]
     },
     {
@@ -35,7 +41,14 @@
       "contributions": [
         "code",
         "design",
-        "doc"
+        "doc",
+        "bug",
+        "userTesting",
+        "ideas",
+        "infra",
+        "maintenance",
+        "test",
+        "question"
       ]
     }
   ],
@@ -44,5 +57,5 @@
   "projectOwner": "cpp-lln-lab",
   "repoType": "github",
   "repoHost": "https://github.com",
-  "skipCi": true
+  "skipCi": false
 }

.travis.yml

@@ -44,7 +44,7 @@ before_script:
 jobs:
   include:
     - name: "BIDS validator: create and check dataset"
-      script: octave $OCTFLAGS --eval "test_makeRawDataset" && bids-validator `pwd`/output/rawdata/ --ignoreNiftiHeaders
+      script: octave $OCTFLAGS --eval "test_makeRawDataset" && bids-validator `pwd`/output/raw/ --ignoreNiftiHeaders
     - name: "miss_hit: checking code quality"
       script: mh_metric . --ci
     - name: "miss_hit: checking code style"

README.md

@@ -125,6 +125,10 @@ future BEP format in a dedicated eyetracker folder
 `<matches>[_recording-<label>]_stim.tsv.gz`
 `<matches>[_recording-<label>]_stim.json`
 
+### change log
+
+ <!-- 93b4c584bf22883a3c4f8b9031b70e381deef272 -->
+
 ###  3.3. <a name='Contributors'></a>Contributors ✨
 
 Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/docs/en/emoji-key)):
@@ -134,9 +138,9 @@ Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/d
 <!-- markdownlint-disable -->
 <table>
   <tr>
-    <td align="center"><a href="https://github.com/CerenB"><img src="https://avatars1.githubusercontent.com/u/10451654?v=4" width="100px;" alt=""/><br /><sub><b>CerenB</b></sub></a><br /><a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=CerenB" title="Code">💻</a> <a href="#design-CerenB" title="Design">🎨</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=CerenB" title="Documentation">📖</a></td>
-    <td align="center"><a href="https://github.com/marcobarilari"><img src="https://avatars3.githubusercontent.com/u/38101692?v=4" width="100px;" alt=""/><br /><sub><b>marcobarilari</b></sub></a><br /><a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=marcobarilari" title="Code">💻</a> <a href="#design-marcobarilari" title="Design">🎨</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=marcobarilari" title="Documentation">📖</a></td>
-    <td align="center"><a href="https://remi-gau.github.io/"><img src="https://avatars3.githubusercontent.com/u/6961185?v=4" width="100px;" alt=""/><br /><sub><b>Remi Gau</b></sub></a><br /><a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=Remi-Gau" title="Code">💻</a> <a href="#design-Remi-Gau" title="Design">🎨</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=Remi-Gau" title="Documentation">📖</a></td>
+    <td align="center"><a href="https://github.com/CerenB"><img src="https://avatars1.githubusercontent.com/u/10451654?v=4" width="100px;" alt=""/><br /><sub><b>CerenB</b></sub></a><br /><a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=CerenB" title="Code">💻</a> <a href="#design-CerenB" title="Design">🎨</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=CerenB" title="Documentation">📖</a> <a href="#userTesting-CerenB" title="User Testing">📓</a> <a href="#ideas-CerenB" title="Ideas, Planning, & Feedback">🤔</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/issues?q=author%3ACerenB" title="Bug reports">🐛</a></td>
+    <td align="center"><a href="https://github.com/marcobarilari"><img src="https://avatars3.githubusercontent.com/u/38101692?v=4" width="100px;" alt=""/><br /><sub><b>marcobarilari</b></sub></a><br /><a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=marcobarilari" title="Code">💻</a> <a href="#design-marcobarilari" title="Design">🎨</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=marcobarilari" title="Documentation">📖</a> <a href="#userTesting-marcobarilari" title="User Testing">📓</a> <a href="#ideas-marcobarilari" title="Ideas, Planning, & Feedback">🤔</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/issues?q=author%3Amarcobarilari" title="Bug reports">🐛</a></td>
+    <td align="center"><a href="https://remi-gau.github.io/"><img src="https://avatars3.githubusercontent.com/u/6961185?v=4" width="100px;" alt=""/><br /><sub><b>Remi Gau</b></sub></a><br /><a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=Remi-Gau" title="Code">💻</a> <a href="#design-Remi-Gau" title="Design">🎨</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=Remi-Gau" title="Documentation">📖</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/issues?q=author%3ARemi-Gau" title="Bug reports">🐛</a> <a href="#userTesting-Remi-Gau" title="User Testing">📓</a> <a href="#ideas-Remi-Gau" title="Ideas, Planning, & Feedback">🤔</a> <a href="#infra-Remi-Gau" title="Infrastructure (Hosting, Build-Tools, etc)">🚇</a> <a href="#maintenance-Remi-Gau" title="Maintenance">🚧</a> <a href="https://github.com/cpp-lln-lab/CPP_BIDS/commits?author=Remi-Gau" title="Tests">⚠️</a> <a href="#question-Remi-Gau" title="Answering Questions">💬</a></td>
   </tr>
 </table>
 

manualTests/miss_hit.cfg

@@ -1,3 +1,10 @@
+# style guide (https://florianschanda.github.io/miss_hit/style_checker.html)
 line_length: 100
-regex_function_name: "(test_[a-z]+)(([A-Z]){1}[A-Za-z]+)*"
-suppress_rule: "copyright_notice"
+regex_function_name: "((test_[a-z]+)|[a-z]+)(([A-Z]){1}[A-Za-z]+)*"
+suppress_rule: "copyright_notice"
+
+# metrics limit for the code quality (https://florianschanda.github.io/miss_hit/metrics.html)
+metric "cnest": limit 4
+metric "file_length": limit 500
+metric "cyc": limit 15
+metric "parameters": limit 5

miss_hit.cfg

@@ -1,4 +1,4 @@
-# styly guide (https://florianschanda.github.io/miss_hit/style_checker.html)
+# style guide (https://florianschanda.github.io/miss_hit/style_checker.html)
 line_length: 100
 regex_function_name: "[a-z]+(([A-Z]){1}[A-Za-z]+)*"
 suppress_rule: "copyright_notice"

src/checkCFG.m

@@ -12,7 +12,9 @@
 
     %% list the defaults to set
 
-    fieldsToSet.verbose = false;
+    fieldsToSet.verbose = 0;
+
+    fieldsToSet.useGUI = false;
 
     fieldsToSet.fileName.task = '';
     fieldsToSet.fileName.zeroPadding = 3;

src/convertSourceToRaw.m

@@ -10,7 +10,7 @@ function convertSourceToRaw(cfg)
     % Only covers func folder at the moment
 
     sourceDir = fullfile(cfg.dir.output, 'source');
-    rawDir = fullfile(cfg.dir.output, 'rawdata');
+    rawDir = fullfile(cfg.dir.output, 'raw');
 
     % add dummy README and CHANGE file
     copyfile(fullfile( ...

src/createFilename.m

@@ -105,7 +105,7 @@
         'echo', ...
         'phaseEncodingDirection', ...
         'reconstruction', ...
-        'recording', ...
+        'recording' ...
         };
 
     targetFields = { ...
@@ -114,7 +114,7 @@
         'echo', ...
         'dir', ...
         'rec', ...
-        'recording', ...
+        'recording' ...
         };
 
     for iField = 1:numel(fields2Check)
@@ -146,10 +146,10 @@
     pattern = cfg.fileName.pattern;
 
     runSuffix = cfg.fileName.suffix.run;
-    acqSuffix = cfg.fileName.suffix.acquisition ;
-    ceSuffix = cfg.fileName.suffix.contrastEnhancement ;
-    dirSuffix = cfg.fileName.suffix.phaseEncodingDirection ;
-    recSuffix = cfg.fileName.suffix.reconstruction ;
+    acqSuffix = cfg.fileName.suffix.acquisition;
+    ceSuffix = cfg.fileName.suffix.contrastEnhancement;
+    dirSuffix = cfg.fileName.suffix.phaseEncodingDirection;
+    recSuffix = cfg.fileName.suffix.reconstruction;
     echoSuffix = cfg.fileName.suffix.echo;
     recordingSuffix = cfg.fileName.suffix.recording;
 
@@ -201,7 +201,7 @@
 
 function talkToMe(cfg)
 
-    if cfg.verbose
+    if cfg.verbose > 0
 
         fprintf(1, '\nData will be saved in this directory:\n\t%s\n', ...
             fullfile(cfg.dir.outputSubject, cfg.fileName.modality));

src/miss_hit.cfg

@@ -1,3 +1,10 @@
+# style guide (https://florianschanda.github.io/miss_hit/style_checker.html)
 line_length: 100
 regex_function_name: "[a-z]+(([A-Z]){1}[A-Za-z]+)*"
-suppress_rule: "copyright_notice"
+suppress_rule: "copyright_notice"
+
+# metrics limit for the code quality (https://florianschanda.github.io/miss_hit/metrics.html)
+metric "cnest": limit 4
+metric "file_length": limit 500
+metric "cyc": limit 15
+metric "parameters": limit 6

src/saveEventsFile.m

@@ -219,14 +219,14 @@ function printHeaderExtraColumns(logFile)
             warning('saveEventsFile:missingData', ...
                 'Missing some %s data for this event.', namesExtraColumns{iExtraColumn});
 
-            if cfg.verbose
+            if cfg.verbose > 1
                 disp(logFile(iEvent));
             end
 
         elseif ~ischar(data) && all(isnan(data))
             warning('Missing %s data for this event.', namesExtraColumns{iExtraColumn});
 
-            if cfg.verbose
+            if cfg.verbose > 1
                 disp(logFile(iEvent));
             end
         end
@@ -297,11 +297,11 @@ function printHeaderExtraColumns(logFile)
 
         else
 
-            printData(logFile(1).fileID, onset);
-            printData(logFile(1).fileID, duration);
-            printData(logFile(1).fileID, trial_type);
+            printData(logFile(1).fileID, onset, cfg);
+            printData(logFile(1).fileID, duration, cfg);
+            printData(logFile(1).fileID, trial_type, cfg);
 
-            printExtraColumns(logFile, iEvent);
+            printExtraColumns(logFile, iEvent, cfg);
 
             fprintf(logFile(1).fileID, '\n');
             fprintf(1, '\n');
@@ -311,7 +311,7 @@ function printHeaderExtraColumns(logFile)
 
 end
 
-function printExtraColumns(logFile, iEvent)
+function printExtraColumns(logFile, iEvent, cfg)
     % loops through the extra columns and print them
 
     namesExtraColumns = returnNamesExtraColumns(logFile);
@@ -320,26 +320,32 @@ function printExtraColumns(logFile, iEvent)
 
         data = logFile(iEvent).(namesExtraColumns{iExtraColumn});
 
-        printData(logFile(1).fileID, data);
+        printData(logFile(1).fileID, data, cfg);
 
     end
 
 end
 
-function printData(output, data)
+function printData(output, data, cfg)
     % write char
     % for numeric data we replace any nan by n/a
     if ischar(data)
         fprintf(output, '%s\t', data);
-        fprintf(1, '%s\t', data);
+        if cfg.verbose > 0
+            fprintf(1, '%s\t', data);
+        end
     else
         for i = 1:numel(data)
             if isnan(data(i))
                 fprintf(output, '%s\t', 'n/a');
-                fprintf(1, '%s\t', 'n/a');
+                if cfg.verbose > 0
+                    fprintf(1, '%s\t', 'n/a');
+                end
             else
                 fprintf(output, '%f\t', data(i));
-                fprintf(1, '%f\t', data(i));
+                if cfg.verbose > 0
+                    fprintf(1, '%f\t', data(i));
+                end
             end
         end
     end
@@ -384,7 +390,7 @@ function errorSaveEventsFile(identifier)
 
 function talkToMe(cfg, logFile)
 
-    if cfg.verbose
+    if cfg.verbose > 0
 
         fprintf(1, '\nData were saved in this file:\n\n%s\n\n', ...
             fullfile( ...