AMP-SCZ / qqc

Checking signals in the AMP-SCZ phantom
https://phantom-check.readthedocs.io/en/latest/
Apache License 2.0
0 stars 1 forks source link

`check_mri_date` updated #24

Closed kcho closed 1 year ago

kcho commented 1 year ago

check_mri_date is updated to

To test the function

cd tests/ampscz_asana/lib
pytest test_qc.py -s -k /tests/ampscz_asana/lib
kcho commented 1 year ago

Looks good! Although, the .group(1) means that it will only check the first file that matches that pattern right? I'm not sure if maybe you want to add a check for duplicates somewhere (unless that's covered by another part of the code).

Thanks Owen! .group(1) gets the string matched by the re.search, that is inside the parenthesis in the pattern.

eg)

re.search(r'ABC(DEF)', x).group(1) gets DEF if x contains 'ABCDEF'.