Bepv01

[sifaoufatai]

To create a BIDS (Brain Imaging Data Structure) structure using YAML files from the BIDS specification and download an experiment,

[pep8speaks]

Hello @sifaoufatai! Thanks for updating this PR.

• In the file bep32v01/BidsDirectoryStructure.py:

Line 41:101: E501 line too long (102 > 100 characters)
Line 55:101: E501 line too long (194 > 100 characters)

• In the file bep32v01/BidsFilestructure.py:

Line 3:1: E302 expected 2 blank lines, found 1
Line 38:101: E501 line too long (102 > 100 characters)

• In the file bep32v01/Createdirectory.py:

Line 9:1: E302 expected 2 blank lines, found 1

• In the file bep32v01/launch.py:

Line 27:101: E501 line too long (104 > 100 characters)

• In the file elab_bridge/Test_export_bids.py:

Line 9:101: E501 line too long (118 > 100 characters)
Line 16:101: E501 line too long (105 > 100 characters)

Comment last updated at 2024-04-05 13:04:13 UTC

[killianrochet]

Hello @sifaoufatai! Thanks for updating this PR.

• In the file bep32v01/BidsDirectoryStructure.py:

Line 41:101: E501 line too long (102 > 100 characters)
Line 55:101: E501 line too long (194 > 100 characters)

• In the file bep32v01/BidsFilestructure.py:

Line 3:1: E302 expected 2 blank lines, found 1
Line 38:101: E501 line too long (102 > 100 characters)

• In the file bep32v01/Createdirectory.py:

Line 9:1: E302 expected 2 blank lines, found 1

• In the file bep32v01/launch.py:

Line 27:101: E501 line too long (104 > 100 characters)

• In the file elab_bridge/Test_export_bids.py:

Line 9:101: E501 line too long (118 > 100 characters)
Line 16:101: E501 line too long (105 > 100 characters)

Comment last updated at 2024-04-05 13:04:13 UTC

@sifaoufatai penses à prendre en compte les recommandations de PEP8 et de faire les modifs en conséquences

[killianrochet]

Pense a mettre tout les prints en Anglais, c'est important !

[killianrochet]

@sifaoufatai L'utilisation des schémas pour BIDS est super, ça respecte ce que l'on veut. En revanche, à voir au niveau discussion mais je pense qu'il est mieux d'aller load le github de BIDS et des spécs plutôt que de copier à la mano les ressources dans le github du NIT. Je m'explique, pour une question de maintenabilité (quand des specs seront add etc etc), question lourdeur de code aussi c'est assez lourd, question duplicata aussi. C'est mon point de vu, à voir ce que @SylvainTakerkart en pense aussi. Mais je pense qu'il sera mieux de juste aller récupérer les ressources sur le github de BIDS plutot que de les copier