Skip to content

Windows example spm - #315

Merged
tiborauer merged 10 commits into
automaticanalysis:masterfrom
AljenU:windows_example_spm
Jan 11, 2022
Merged

Windows example spm#315
tiborauer merged 10 commits into
automaticanalysis:masterfrom
AljenU:windows_example_spm

Conversation

@AljenU

@AljenU AljenU commented Dec 16, 2021

Copy link
Copy Markdown
Collaborator

Changes to have tutorial_2_SPM_CH30 run out-of-the-box, and also on a Windows based Matlab Parallel Server cluster.

Partial fix for #313
Changes for #276
Changes for #256

Escape backslash characters in the windows paths, to prevent them from being
interpreted as special character identifiers in printf functions

Partial fix for issue automaticanalysis#276
On windows, always use copyfile fow now. See the related issue for thoughts and
further options.

Partial fix for issue automaticanalysis#313
Make the comments more in line with tutorial_1_aa_setup, and remove outdated
comment on not-gzipped data.

Add automatic data download, and update what variables are set. With a minimal
correct user parameter file, the tutorial will now run without needing any user edits.
This also enables automatic testing of this tutorial function.
Escape backslash characters in possible windows paths, to prevent them from being
interpreted as special character identifiers in printf functions

Partial fix for issue automaticanalysis#276
A Parallel Server MJS cluster may run tasks under a different user, with different drive mappings. Convert mapped drive paths to unc paths to avoid problems from this different mapping

Note that this does not solve an problems caused by the different user having different access rights to files and directories
Comment thread developer/tests/@testExamples/testExamples.m
Comment thread extrafunctions/get_drive_mappings.m

@tiborauer tiborauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, see my specific comments.

@tiborauer
tiborauer merged commit a4945b5 into automaticanalysis:master Jan 11, 2022
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

Successfully merging this pull request may close these issues.

2 participants