-
Notifications
You must be signed in to change notification settings - Fork 1
add code to create raw tsv and make a markdown table + add submodules #2
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
Conversation
Hello @marcobarilari! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2023-11-08 09:11:57 UTC |
@Remi-Gau will merge tomorrow morning (around 10:15) to have some basic stuff to show around. However, if you think you want to review this first one to start with the right step I am happy to leave it open for a little while more. |
src/datasets_raw.md
Outdated
@@ -0,0 +1,25 @@ | |||
| name | nb_subjects | has_participant_tsv | has_participant_json | participant_columns | has_phenotype_dir | modalities | sessions | tasks | fmriprep | freesurfer | mriqc | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for the moment I would drop the columns :
- has_participant_tsv
- has_participant_json
- has_phenotype_dir
- participant_columns
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
drop some of the columns we do not care about
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
you probably want to:
|
also if you want to jsut reformat your code to get less complaints like this one, pip install black
black . |
will push a couple of changes and merge |
add code to create raw tsv and make a markdown table + add submodules (Sourcery refactored)
No description provided.