Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comments on main_script_1st_level.py #1

Open
me-pic opened this issue Jun 20, 2022 · 0 comments
Open

Comments on main_script_1st_level.py #1

me-pic opened this issue Jun 20, 2022 · 0 comments

Comments

@me-pic
Copy link
Collaborator

me-pic commented Jun 20, 2022

Pour la création du folder (L40:L42): possible de simplement créer un fichier avec la date et l'heure si le fichier spécifié existe déjà

Selon les préférences: importer tous les modules nécessaires au début du script: L64/73/91/111/125/178

Dans les prints pour éviter de concaténer tes strings avec des +, tu peux simplement utiliser f devant ton string avec les {} pour les variables. Ex avec L94:
print('COMPUTING design matrix under name : ' + DM_name + ' for subject ' + subj_name)
vs
print(f'COMPUTING design matrix under name : {DM_name} for {subject subj_name}')
Tu pourrais également utiliser la même logique pour ta variable fmri_img_name (L114)

Pour des blocs de code tu peux utiliser les """ blablabla """ au lieu de mettre des # pour commenter chaque ligne. P.ex. pour les lignes 136:145 tu pourrais simplement mettre un """ à la ligne 135 et un """ à la ligne 146

Normal qu'il n'y ait rien dans le write à L192

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant