Skip to content

Conversation

@maxpietsch
Copy link
Member

Fix to population_template which currently does not handle whitespace characters in the unique identifier parts of the input files. This fix replaces whitespace with '_' as internally, population_template requires filenames in its scratch directory to be whitespace free.

Likely fix to https://community.mrtrix.org/t/fba-population-template-error-uid-contains-whitespace/4229/4

➜  mt_reg_test ls -l odfA3
total 3952
-rw-r--r--  1 mp  staff  699956 26 Oct 14:32 odf 0.mif.gz
-rw-r--r--  1 mp  staff  642369 26 Oct 14:32 odf 2.mif.gz
-rw-r--r--  1 mp  staff  678072 26 Oct 14:33 odf subject 1.mif.gz

➜  mt_reg_test population_template odfA3 out.mif -scratch /tmp/scratch -debug
population_template: [DEBUG] path.from_user() (from population_template:291): out.mif -> /Users/mp/data/mt_reg_test/out.mif
population_template: [DEBUG] execute() (from app.py:197): 	contrast: ., template: /Users/mp/data/mt_reg_test/out.mif, suffix: _c0

population_template: Generating a population-average template from 3 input images
population_template: [WARNING] no masks input. Use input masks to reduce computation time and improve robustness
population_template: common_postfix: ['.mif.gz']
population_template: common_prefix: ['odf ']
population_template: xcontrast_xsubject_pre_postfix: ['odf ', '']
population_template: [DEBUG] path_to_file_uid() (from population_template:519): UID mapping: "odf 0.mif.gz" --> "0"
population_template: [DEBUG] path_to_file_uid() (from population_template:519): UID mapping: "odf 2.mif.gz" --> "2"
population_template: [DEBUG] path_to_file_uid() (from population_template:519): UID mapping: "odf subject 1.mif.gz" --> "subject_1"
population_template: uids by contrast:[['0', '2', 'subject_1']]
population_template: [DEBUG] path.from_user() (from population_template:532): odfA3 -> /Users/mp/data/mt_reg_test/odfA3
population_template: [DEBUG] path.from_user() (from population_template:532): odfA3 -> /Users/mp/data/mt_reg_test/odfA3
population_template: [DEBUG] path.from_user() (from population_template:532): odfA3 -> /Users/mp/data/mt_reg_test/odfA3
population_template: performing leave-one-out registration

@maxpietsch maxpietsch added the bug label Oct 26, 2020
@maxpietsch maxpietsch force-pushed the population_template_space_fix branch from ae035e7 to 27fba28 Compare January 20, 2021 09:25
@maxpietsch maxpietsch requested a review from a team January 20, 2021 10:16
@maxpietsch maxpietsch added this to the 3.0.3 hotfix milestone Apr 6, 2021
@Lestropie Lestropie modified the milestones: 3.0.4 hotfix, 3.0.3 hotfix Jun 25, 2021
@jdtournier jdtournier enabled auto-merge June 30, 2021 10:13
@jdtournier jdtournier merged commit 87ff24f into master Jun 30, 2021
@jdtournier jdtournier deleted the population_template_space_fix branch June 30, 2021 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants