search

cpp-lln-lab / bidspm

an SPM centric BIDS app
https://bidspm.readthedocs.io/en/latest/general_information.html
GNU General Public License v3.0
7 stars 14 forks source link

[BUG] Cannot set raw data to read only when running docker container #1185

Closed monique2208 closed 9 months ago

monique2208 commented 9 months ago

Is there an existing issue for this?

Operating system

Operating system version

Windows 11

SPM 12 version

Platform

Platform version

Latest docker container

bidspm version

v3.1.0

bidspm branch / commit number

No response

Expected Behavior

I am used to setting specifying that the raw data in bids should be read-only, this seems like good practice, but when I run the stats action using bidspm this results in an error. I ran the following:

docker run --rm -it \
  -v .\ds003126_3sub_test:/bids:ro \
  -v .\ds003126_3sub_test\derivatives\bidspm-preproc:/prep:ro \
  -v .\ds003126_3sub_test:/out \
  -v .\ds003126_3sub_test\derivatives\models\model-defaultRead_smdl.json:/model:ro \
  cpplab/bidspm /bids /out dataset --action stats \
  --task read --model /model --preproc_dir /prep \
  --space MNI152NLin2009cAsym --skip_validation

If I omit the ro flag when binding the directory, it produces the statistical result as I expect. As far as I can tell nothing gets added or changed in the raw data, it seems that some onset file is written temporarily into the raw data.

BIDS stats model

No response

Current Behavior & Error message

[15:16:27] INFO     bidspm - [15:16:27] - INFO - Running statistics.                                                                  bidspm.py:261

Running the following command:

 bidspm();
 bidspm('/bids', ...
                         '/out', ...
                         'subject', ...
                         'action', 'stats', ...
                         'verbosity', 2, ...
                         'space', { 'MNI152NLin2009cAsym' }, ...
                         'task', { 'read' }, ...
                         'fwhm', 6.0, ...
                         'skip_validation', true, ...
                         'preproc_dir', '/prep', ...
                         'model_file', '/model');
 exit;

warning: function /opt/spm12/gui_mainfcn.m shadows a core library function
warning: called from
    /root/.octaverc at line 2 column 3

MarsBaR analysis functions prepended to path
Loaded MarsBaR defaults from base defaults
warning: Unable to open display.
warning: called from
    spm at line 656 column 9
    spm at line 577 column 4
    marsbar at line 458 column 6
    initCppRoi at line 27 column 7
    run at line 78 column 7
    bidspm>initBidspm at line 454 column 5
    bidspm>executeAction at line 91 column 7
    bidspm at line 77 column 5

warning: isdir is obsolete; use isfolder or dir_in_loadpath instead

Copying atlas "HCPex" to spm atlas directory:
        /opt/spm12/atlas

Copying atlas "AAL" to spm atlas directory:
        /opt/spm12/atlas
Correct matlab/octave versions and added to the path!
Adding code to the path.
checking MATLAB / Octave version.

[15:16:28] bidspm - INFO                                checkDependencies
Checking dependencies

[15:16:28] bidspm - INFO                                checkDependencies
 Using SPM12 7771
Defaults settings have been modified by file(s):
  /home/neuro/bidspm/src/defaults/spm_my_defaults.m
Modified fields: ui mat stats realign
____________________________________________________________
____________________________________________________________

______  _____ ______  _____ ______ ___  ___
| ___ \|_   _||  _  \/  ___|| ___ \|  \/  |
| |_/ /  | |  | | | |\ `--. | |_/ /| .  . |
| ___ \  | |  | | | | `--. \|  __/ | |\/| |
| |_/ / _| |_ | |/ / /\__/ /| |    | |  | |
\____/  \___/ |___/  \____/ \_|    \_|  |_/

Thank you for using bidspm - version v3.0.0

Current list of contributors includes

        - Jeanne Caron-Guyon
        - Iqra Shahzad
        - Olivier Collignon
        - Ying Yang
        - Stefania Mattioni
        - Remi Gau
        - Marco Barilari
        - Christine Chouinard-Leclaire
        - Ane Gurtubay
        - Federica Falagiarda
        - Márcia Nunes
        - Manon Chateaux
        - Michèle MacLean
        - Filippo Cerpelloni
        - Ceren Battal
        - Mohamed Rezk
        - Why not be the next?

Please cite using the following DOI:
 https://doi.org/10.5281/zenodo.3554331.

For bug report & suggestions see our github repo:
 https://github.com/cpp-lln-lab/bidspm

____________________________________________________________
____________________________________________________________

[15:16:33] bidspm - INFO
bidspm already initialized

[15:16:33] bidspm - INFO                                saveOptions
Options saved in: /home/neuro/options/options_task-read_2024-01-23T15-16.json
fatal: not a git repository (or any of the parent directories): .git
fatal: not a git repository (or any of the parent directories): .git
warning:
Could not find a corresponding Node with
  Name: "dataset_level"; Level: ""

  Available nodes:
        - Name: "run"; Level: "Run"
        - Name: "subject"; Level: "Subject"
        - Name: "dataset"; Level: "Dataset"

warning: Could not find key 'version'
warning: called from
    get_key at line 74 column 13
    renderer at line 159 column 19
    renderer at line 222 column 25
    renderer at line 281 column 22
    octache at line 79 column 12
    boilerplate at line 149 column 10
    bidspm>stats at line 343 column 5
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

warning: Could not find key 'designType'
warning: called from
    get_key at line 74 column 13
    renderer at line 159 column 19
    octache at line 79 column 12
    boilerplate at line 149 column 10
    bidspm>stats at line 343 column 5
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

[15:16:33] bidspm - INFO                                boilerplate
## fMRI statistical analysis

The fMRI data were analysed with bidspm (v3.0.0; https://github.com/cpp-lln-lab/bidspm; DOI: https://doi.org/10.5281/zenodo.3554331 - [@bidspm])
using statistical parametric mapping
(SPM12 - 7771; Wellcome Center for Neuroimaging, London, UK;
https://www.fil.ion.ucl.ac.uk/spm; RRID:SCR_007037)
using Octave 6.4.0
on a x86_64-pc-linux-gnu computer ({{version}})

The input data were the preprocessed BOLD images in MNI152NLin2009cAsym space for the task " read ".

### Run / subject level analysis

At the subject level, we performed a mass univariate analysis with a linear
regression at each voxel of the brain, using generalized least squares with a
global  FAST  model to account for temporal auto-correlation
 [@Corbin2018] and a drift fit with discrete cosine transform basis ( 125 seconds cut-off).

Image intensity scaling was done run-wide before statistical modeling such that
the mean image would have a mean intracerebral intensity of 100.

We modeled the fMRI experiment in a  {{designType}}  design with regressors
entered into the run-specific design matrix. The onsets
were convolved with SPM canonical hemodynamic response function (HRF)
 for the conditions:
  - `Fixation`,
 - `PH`,
 - `VerbalFeedback`,
 - `W`,
 .

 Nuisance covariates included:

 - `1`,
 - `trans_?`,
 - `rot_?`,
 - `non_steady_state_outlier*`,
 - `motion_outlier*`,

to account for residual motion artefacts,
to censor any time points flagged as outlier,
 .

 ### Group level analysis

Contrast for the following conditions were passed as summary statistics for a group level analysis:

 ## References

This method section was automatically generated using bidspm
(v3.0.0; https://github.com/cpp-lln-lab/bidspm; DOI: https://doi.org/10.5281/zenodo.3554331)
and octache (https://github.com/Remi-Gau/Octache).

[15:16:34] bidspm - INFO                                getData
FOR TASK(s): read

Indexing dataset:
        /prep
warning:
Missing: /prep/participants.tsv

To silence this warning, consider adding a "participants.tsv" to your dataset.
See the function: bids.util.create_participants_tsv
See also this section of the BIDS specification:
        https://bids-specification.readthedocs.io/en/latest/03-modality-agnostic-files.html#participants-file
 Indexing subject: sub-047EPKL014007 [..]
 Indexing subject: sub-047EPKL014008 [..]
 Indexing subject: sub-047EPKL014010 [..]

Indexing dataset:
        /bids
warning: genvarname is obsolete; use matlab.lang.makeValidName or matlab.lang.makeUniqueStrings instead
 Indexing subject: sub-047EPKL014008 [..]
 Indexing subject: sub-047EPKL014010 [..]
 Indexing subject: sub-047EPKL014020 [..]

[15:16:53] bidspm - INFO                                getData
WILL WORK ON SUBJECTS
        - 047EPKL014007
        - 047EPKL014008
        - 047EPKL014010

[15:16:53] bidspm - INFO                                printWorkflowName
WORKFLOW: SUBJECT LEVEL GLM

[15:16:53] bidspm - INFO                                bidsFFX

 PROCESSING NODE: run

Indexing dataset:
        /out/derivatives/bidspm-stats
warning:
Missing: /out/derivatives/bidspm-stats/participants.tsv

To silence this warning, consider adding a "participants.tsv" to your dataset.
See the function: bids.util.create_participants_tsv
See also this section of the BIDS specification:
        https://bids-specification.readthedocs.io/en/latest/03-modality-agnostic-files.html#participants-file
warning:
No subjects found in BIDS directory: '/out/derivatives/bidspm-stats'

[15:16:53] bidspm - INFO                                printProcessingSubject
PROCESSING SUBJECT No.: 1 SUBJECT LABEL : 047EPKL014007

[15:16:54] bidspm - INFO                                printBatchName
 BUILDING JOB: specify subject level fmri model
warning: Non-existent field "SliceTiming" for metadata.
warning: called from
    query>update_result at line 459 column 15
    query>perform_query at line 395 column 14
    query at line 199 column 10
    getAndCheckSliceOrder at line 36 column 15
    setBatchSubjectLevelGLMSpec>returnSliceOrder at line 206 column 16
    setBatchSubjectLevelGLMSpec at line 46 column 14
    bidsFFX>setAction at line 235 column 19
    bidsFFX at line 109 column 21
    bidspm>stats at line 364 column 13
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

warning: Non-existent field "SliceTiming" for metadata.
warning: called from
    query>update_result at line 459 column 15
    query>perform_query at line 395 column 14
    query at line 199 column 10
    getAndCheckSliceOrder at line 36 column 15
    setBatchSubjectLevelGLMSpec>returnSliceOrder at line 206 column 16
    setBatchSubjectLevelGLMSpec at line 46 column 14
    bidsFFX>setAction at line 235 column 19
    bidsFFX at line 109 column 21
    bidspm>stats at line 364 column 13
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

warning: Non-existent field "SliceTiming" for metadata.
warning: called from
    query>update_result at line 459 column 15
    query>perform_query at line 395 column 14
    query at line 199 column 10
    getAndCheckSliceOrder at line 36 column 15
    setBatchSubjectLevelGLMSpec>returnSliceOrder at line 206 column 16
    setBatchSubjectLevelGLMSpec at line 46 column 14
    bidsFFX>setAction at line 235 column 19
    bidsFFX at line 109 column 21
    bidspm>stats at line 364 column 13
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

warning:
[15:16:54] bidspm - WARNING                             setBatchSubjectLevelGLMSpec

Slice timing information was missing for at least one run,
or was inconsistent across runs.
Will be using the number of slices as the number of bins
for temporal upsampling before convolution.
If your data was processed with fMRIprep < 20.2.4, this is expected.

warning:
[15:16:54] bidspm - WARNING                             overwriteDir

overwriting directory:
        /out/derivatives/bidspm-stats/sub-047EPKL014007/task-read_space-MNI152NLin2009cAsym_FWHM-6

[15:16:54] bidspm - INFO                                setBatchSubjectLevelGLMSpec
 output dir:
        /out/derivatives/bidspm-stats/sub-047EPKL014007/task-read_space-MNI152NLin2009cAsym_FWHM-6

[15:16:54] bidspm - INFO                                setBatchSubjectLevelGLMSpec
 Processing run 01

[15:16:54] bidspm - INFO                                getBoldFilenameForFFX
  Bold file(s):
        - /prep/sub-047EPKL014007/ses-1/func/sub-047EPKL014007_ses-1_task-read_run-01_space-MNI152NLin2009cAsym_desc-smth6_bold.nii

warning:
[15:16:54] bidspm - WARNING                             setBatchSubjectLevelGLMSpec
No events.tsv file found in:
        /bids
for filter:
        - extension: {.tsv}
        - modality: {func}
        - prefix: {}
        - sub: {^047EPKL014007$}
        - suffix: {events}
        - task: {read}
        - ses: {1}
        - run: {01}

[15:16:55] bidspm - INFO                                setBatchSubjectLevelGLMSpec
 Processing run 02

[15:16:55] bidspm - INFO                                getBoldFilenameForFFX
  Bold file(s):
        - /prep/sub-047EPKL014007/ses-1/func/sub-047EPKL014007_ses-1_task-read_run-02_space-MNI152NLin2009cAsym_desc-smth6_bold.nii

warning:
[15:16:55] bidspm - WARNING                             setBatchSubjectLevelGLMSpec
No events.tsv file found in:
        /bids
for filter:
        - extension: {.tsv}
        - modality: {func}
        - prefix: {}
        - sub: {^047EPKL014007$}
        - suffix: {events}
        - task: {read}
        - ses: {1}
        - run: {02}

[15:16:56] bidspm - INFO                                setBatchSubjectLevelGLMSpec
 Processing run 03

[15:16:56] bidspm - INFO                                getBoldFilenameForFFX
  Bold file(s):
        - /prep/sub-047EPKL014007/ses-1/func/sub-047EPKL014007_ses-1_task-read_run-03_space-MNI152NLin2009cAsym_desc-smth6_bold.nii

warning:
[15:16:56] bidspm - WARNING                             setBatchSubjectLevelGLMSpec
No events.tsv file found in:
        /bids
for filter:
        - extension: {.tsv}
        - modality: {func}
        - prefix: {}
        - sub: {^047EPKL014007$}
        - suffix: {events}
        - task: {read}
        - ses: {1}
        - run: {03}

warning:
[15:16:58] bidspm - WARNING                             getInclusiveMask
More than 1 mask image found for
        - desc: {brain}
        - suffix: {mask}
        - sub: {^047EPKL014007$}
        - space: {MNI152NLin2009cAsym}.

Taking the first one:
        /prep/sub-047EPKL014007/ses-1/anat/sub-047EPKL014007_ses-1_space-MNI152NLin2009cAsym_desc-brain_mask.nii

from:
        - /prep/sub-047EPKL014007/ses-1/anat/sub-047EPKL014007_ses-1_space-MNI152NLin2009cAsym_desc-brain_mask.nii
        - /prep/sub-047EPKL014007/ses-1/func/sub-047EPKL014007_ses-1_task-read_run-01_space-MNI152NLin2009cAsym_desc-brain_mask.nii
        - /prep/sub-047EPKL014007/ses-1/func/sub-047EPKL014007_ses-1_task-read_run-02_space-MNI152NLin2009cAsym_desc-brain_mask.nii
        - /prep/sub-047EPKL014007/ses-1/func/sub-047EPKL014007_ses-1_task-read_run-03_space-MNI152NLin2009cAsym_desc-brain_mask.nii

[15:16:58] bidspm - INFO                                printBatchName
 BUILDING JOB: print figure

[15:16:58] bidspm - INFO                                printBatchName
 BUILDING JOB: estimate subject level fmri model

[15:16:58] bidspm - INFO                                printBatchName
 BUILDING JOB: print figure

[15:16:58] bidspm - INFO                                saveMatlabBatch
Saving job in:
        /out/derivatives/bidspm-stats/jobs/read
fatal: not a git repository (or any of the parent directories): .git
fatal: not a git repository (or any of the parent directories): .git

------------------------------------------------------------------------
23-Jan-2024 15:17:00 - Running job #1
------------------------------------------------------------------------
23-Jan-2024 15:17:00 - Running 'fMRI model specification'

SPM12: spm_fMRI_design (v7739)                     15:17:00 - 23/01/2024
========================================================================
Saving fMRI design                      :               ...SPM.mat saved

SPM12: spm_fmri_spm_ui (v7738)                     15:17:00 - 23/01/2024
========================================================================
Mapping files                           :                        ...done
Calculating globals                     :                        ...done
Saving SPM configuration                :               ...SPM.mat saved
Saving SPM configuration                :               ...SPM.mat saved
Completed                               :          15:17:22 - 23/01/2024
23-Jan-2024 15:17:22 - Done    'fMRI model specification'
23-Jan-2024 15:17:22 - Running 'Print figure'
Print error: Figure not found.
23-Jan-2024 15:17:22 - Done    'Print figure'
23-Jan-2024 15:17:22 - Running 'Model estimation'

SPM12: spm_spm (v7738)                             15:17:22 - 23/01/2024
========================================================================

SPM12: spm_est_non_sphericity (v7577)              15:17:25 - 23/01/2024
========================================================================
Chunk   1/1                             :                        ...done
Temporal non-sphericity (49616 voxels)  :             ...ReML estimation
  ReML Block 1
  ReML Iteration 1                      :        ...5.528186e+00 [+4.25]
  ReML Iteration 2                      :        ...6.741794e-02 [+4.50]
  ReML Iteration 1                      :        ...5.347189e+00 [+4.75]
  ReML Iteration 2                      :        ...1.128468e-02 [+5.00]
  ReML Block 2
  ReML Iteration 1                      :        ...3.538960e+00 [+4.25]
  ReML Iteration 2                      :        ...2.529848e-02 [+4.50]
  ReML Iteration 1                      :        ...3.281738e+00 [+4.75]
  ReML Iteration 2                      :        ...2.212264e-02 [+5.00]
  ReML Block 3
  ReML Iteration 1                      :        ...3.232951e+00 [+4.25]
  ReML Iteration 2                      :        ...6.685427e-02 [+4.50]
  ReML Iteration 1                      :        ...2.936620e+00 [+4.75]
  ReML Iteration 2                      :        ...6.862466e-02 [+5.00]

SPM12: spm_spm (v7738)                             15:17:44 - 23/01/2024
========================================================================
Chunk   1/1                             :                        ...done
Spatial non-sphericity (over scans)     :                        ...done
Saving SPM.mat                          :                        ...done
Completed                               :          15:18:36 - 23/01/2024
Done
23-Jan-2024 15:18:36 - Done    'Model estimation'
23-Jan-2024 15:18:36 - Running 'Print figure'
Print error: Figure not found.
23-Jan-2024 15:18:36 - Done    'Print figure'
23-Jan-2024 15:18:36 - Done

[15:18:38] bidspm - INFO                                printProcessingSubject
PROCESSING SUBJECT No.: 2 SUBJECT LABEL : 047EPKL014008

[15:18:38] bidspm - INFO                                printBatchName
 BUILDING JOB: specify subject level fmri model
warning: Non-existent field "SliceTiming" for metadata.
warning: called from
    query>update_result at line 459 column 15
    query>perform_query at line 395 column 14
    query at line 199 column 10
    getAndCheckSliceOrder at line 36 column 15
    setBatchSubjectLevelGLMSpec>returnSliceOrder at line 206 column 16
    setBatchSubjectLevelGLMSpec at line 46 column 14
    bidsFFX>setAction at line 235 column 19
    bidsFFX at line 109 column 21
    bidspm>stats at line 364 column 13
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

warning: Non-existent field "SliceTiming" for metadata.
warning: called from
    query>update_result at line 459 column 15
    query>perform_query at line 395 column 14
    query at line 199 column 10
    getAndCheckSliceOrder at line 36 column 15
    setBatchSubjectLevelGLMSpec>returnSliceOrder at line 206 column 16
    setBatchSubjectLevelGLMSpec at line 46 column 14
    bidsFFX>setAction at line 235 column 19
    bidsFFX at line 109 column 21
    bidspm>stats at line 364 column 13
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

warning: Non-existent field "SliceTiming" for metadata.
warning: called from
    query>update_result at line 459 column 15
    query>perform_query at line 395 column 14
    query at line 199 column 10
    getAndCheckSliceOrder at line 36 column 15
    setBatchSubjectLevelGLMSpec>returnSliceOrder at line 206 column 16
    setBatchSubjectLevelGLMSpec at line 46 column 14
    bidsFFX>setAction at line 235 column 19
    bidsFFX at line 109 column 21
    bidspm>stats at line 364 column 13
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

warning:
[15:18:38] bidspm - WARNING                             setBatchSubjectLevelGLMSpec

Slice timing information was missing for at least one run,
or was inconsistent across runs.
Will be using the number of slices as the number of bins
for temporal upsampling before convolution.
If your data was processed with fMRIprep < 20.2.4, this is expected.

warning:
[15:18:39] bidspm - WARNING                             overwriteDir

overwriting directory:
        /out/derivatives/bidspm-stats/sub-047EPKL014008/task-read_space-MNI152NLin2009cAsym_FWHM-6

[15:18:39] bidspm - INFO                                setBatchSubjectLevelGLMSpec
 output dir:
        /out/derivatives/bidspm-stats/sub-047EPKL014008/task-read_space-MNI152NLin2009cAsym_FWHM-6

[15:18:39] bidspm - INFO                                setBatchSubjectLevelGLMSpec
 Processing run 01

[15:18:39] bidspm - INFO                                getBoldFilenameForFFX
  Bold file(s):
        - /prep/sub-047EPKL014008/ses-1/func/sub-047EPKL014008_ses-1_task-read_run-01_space-MNI152NLin2009cAsym_desc-smth6_bold.nii

[15:18:39] bidspm - INFO                                createAndReturnOnsetFile

  Reading the tsv file : /bids/sub-047EPKL014008/ses-1/func/sub-047EPKL014008_ses-1_task-read_run-01_events.tsv

[15:18:39] bidspm - INFO                                convertOnsetTsvToMat
   Condition "Fixation": 42 trials found.

[15:18:39] bidspm - INFO                                convertOnsetTsvToMat
   Condition "PH": 20 trials found.

[15:18:39] bidspm - INFO                                convertOnsetTsvToMat
   Condition "VerbalFeedback": 40 trials found.

[15:18:39] bidspm - INFO                                convertOnsetTsvToMat
   Condition "W": 20 trials found.

fatal: not a git repository (or any of the parent directories): .git
fatal: not a git repository (or any of the parent directories): .git

ERROR LOG SAVED:
        /out/derivatives/bidspm-stats/error_logs/error_2024-01-23T15-18.log
Use it when opening an issue:
        https://github.com/cpp-lln-lab/bidspm/issues/new/choose.

error: save: unable to open output file '/bids/sub-047EPKL014008/ses-1/func/sub-047EPKL014008_ses-1_task-read_run-01_onsets.mat'
error: called from
    convertOnsetTsvToMat at line 175 column 3
    createAndReturnOnsetFile at line 38 column 17
    setBatchSubjectLevelGLMSpec>returnOnsetsFile at line 284 column 17
    setBatchSubjectLevelGLMSpec at line 113 column 23
    bidsFFX>setAction at line 235 column 19
    bidsFFX at line 109 column 21
    bidspm>stats at line 364 column 13
    bidspm>executeAction at line 152 column 7
    bidspm at line 77 column 5

Anything else?

No response

github-actions[bot] commented 9 months ago

Thank you for your issue. Give us a little time to review it.

PS. You might want to check the FAQ if you haven't done so already.

This is an automated reply, generated by FAQtory

Remi-Gau commented 9 months ago

raw data in bids should be read-only, this seems like good practice

indeed that seems like the desirable thing to do

let me check but it seems that for some reason I temporarily save things in the raw dataset and not the derivatives one

Remi-Gau commented 9 months ago

yup this is due to this:

https://github.com/cpp-lln-lab/bidspm/blob/a4a19e15b8cca4bd0e13ac7323b61110838d50c1/src/stats/subject_level/createAndReturnOnsetFile.m#L42-L45

will push a fix to make sure the raw dataset is left pristine

thanks for the bug report

Remi-Gau commented 9 months ago

@monique2208

This should be fixed but I will need to make a new release and push a new docker image so you can make use of it.

Alternatively you should be able to pull the 'unstable' image of bidspm which should correspond to the bidspm that lives on the main branch of this repo.

docker pull cpplab/bidspm:unstable

It can be useful to test that bugs have indeed been fixed.

I just need to make sure that the unstable image is indeed put on docker hub automatically, so don't try it out just yet.

Remi-Gau commented 9 months ago

@all-contributors please add @monique2208 for bug

allcontributors[bot] commented 9 months ago

@Remi-Gau

I've put up a pull request to add @monique2208! :tada: