You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Admittedly, there are too many input files in rule all most of which are not necessary. But even clean_epochs is hard to read.
Solution ideas:
For the templates: use a hierarchy of templates and .format(x=???) so that we don't have som much repeating code. The higher-level templates should have very telling names like base_template_for_run_level_files. Maybe something else. Maybe not repeating ourselves is a bad idea here and explicitly writing out paths is better, IDK.
For the output: let's change the working directory to the BIDS folder or, maybe, its parent and use relative path throughout where neceessary. The paths would still be very long but, I think, it would still help with readability.
The text was updated successfully, but these errors were encountered:
There are two places where paths make it hard to understand what is going on:
Templates in Snakefile:
Snakemake output
A piece of a sample output
```sh [Tue Jul 27 14:34:32 2021] rule clean_epochs: input: E:\Evgenii_Kalenkovich\reproducing-reproducible\01_reproduction\data\bids\derivatives\01_preprocessing\sub-16\ses-meg\meg\sub-16_ses-meg_task-facerecognition_filtered.fif, E:\Evgenii_Kalenkovich\reproducing-reproducible\0 1_reproduction\data\bids\derivatives\01_preprocessing\sub-16\ses-meg\meg\sub-16_ses-meg_task-facerecognition_artifactComponents.npz, E:\Evgenii_Kalenkovich\reproducing-reproducible\01_reproduction\data\bids\derivatives\01_preprocess ing\sub-16\ses-meg\meg\sub-16_ses-meg_task-facerecognition_epo.fif output: E:\Evgenii_Kalenkovich\reproducing-reproducible\01_reproduction\data\bids\derivatives\01_preprocessing\sub-16\ses-meg\meg\sub-16_ses-meg_task-facerecognition_epoCleaned.fif jobid: 672 wildcards: subject_number=16Admittedly, there are too many input files in rule
all
most of which are not necessary. But evenclean_epochs
is hard to read.Solution ideas:
.format(x=???)
so that we don't have som much repeating code. The higher-level templates should have very telling names likebase_template_for_run_level_files
. Maybe something else. Maybe not repeating ourselves is a bad idea here and explicitly writing out paths is better, IDK.The text was updated successfully, but these errors were encountered: