Skip to content

Fix step 9 reference in upload_freesurfer.sh completion message#75

Draft
shawntz with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-fmriprep-step-number
Draft

Fix step 9 reference in upload_freesurfer.sh completion message#75
shawntz with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-fmriprep-step-number

Conversation

Copilot AI commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

The upload_freesurfer.sh script (Step 8) incorrectly referenced Step 7 for running full fMRIPrep workflows. Step 7 doesn't exist—it's Step 9.

Changes

  • Updated completion message to reference Step 9 and ./09-run.sbatch instead of non-existent Step 7 and ./07-run.sbatch

Before

echo "1. Run fMRIPrep step 7 (full workflows) to use edited surfaces"
echo "   ./07-run.sbatch"

After

echo "1. Run fMRIPrep step 9 (full workflows) to use edited surfaces"
echo "   ./09-run.sbatch"

This aligns with the pipeline structure where Steps 7-8 are FreeSurfer editing toolbox utilities, and Step 9 (09-run-fmriprep/) executes the full fMRIPrep workflows.

Original prompt

instructions after running the step 8 upload freesurfer command has the wrong step number for the fmriprep full func workflows step: 9.

Successfully uploaded: 1 subjects

Original Freesurfer outputs backed up on server as:
  {subject}.backup.20251222_134018

Next steps:
1. Run fMRIPrep step 7 (full workflows) to use edited surfaces
   ./07-run.sbatch

please set to merge this into PR #74 (release candidate v0.3.1)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: shawntz <5200266+shawntz@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix step number in fMRIPrep workflow instructions Fix step 9 reference in upload_freesurfer.sh completion message Dec 22, 2025
Copilot AI requested a review from shawntz December 22, 2025 21:51
@shawntz

shawntz commented Jan 1, 2026

Copy link
Copy Markdown
Owner

fizzy

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