From 440fdc240c5c321909fb0496889fca70efe8fcfe Mon Sep 17 00:00:00 2001 From: Tor Wager Date: Sat, 4 Jul 2026 12:51:51 -0400 Subject: [PATCH] tests-walkthroughs CI: check out RobustToolbox (weighted_corrcoef dep) The nightly walkthrough job has failed every night since the suite landed: canlab_test_walkthrough_5_regression errors at section 16 with "Undefined function 'weighted_corrcoef'". That function ships in the canlab/RobustToolbox sibling repo (Robust_stats_functions/), a dependency of plot_correlation_samefig's robust branch, which the regression walkthrough exercises. A normal canlab_toolbox_setup install has RobustToolbox on the path; the CI runner did not. Add a RobustToolbox checkout + addpath(genpath(...)). Not a headless issue -- the fault-tolerant harness already skips display-only sections correctly; this was a missing path dependency. Verified locally: removing RobustToolbox from the path reproduces the exact error; restoring it runs plot_correlation_samefig's robust path cleanly. Co-Authored-By: Claude Opus 4.8 (1M context) --- .github/workflows/tests-walkthroughs.yml | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/.github/workflows/tests-walkthroughs.yml b/.github/workflows/tests-walkthroughs.yml index f7664896..569f2254 100644 --- a/.github/workflows/tests-walkthroughs.yml +++ b/.github/workflows/tests-walkthroughs.yml @@ -39,6 +39,17 @@ jobs: repository: canlab/CANlab_help_examples path: CANlab_help_examples + - name: Checkout RobustToolbox + # Provides weighted_corrcoef (Robust_stats_functions/), a dependency of + # plot_correlation_samefig's robust branch used by the regression + # walkthrough. A normal canlab_toolbox_setup install adds this sibling + # repo to the path; CI must too, or walkthrough 5 errors with + # "Undefined function 'weighted_corrcoef'". + uses: actions/checkout@v4 + with: + repository: canlab/RobustToolbox + path: RobustToolbox + - name: Clone SPM25 run: git clone --depth=1 --branch 25.01.02 https://github.com/spm/spm.git "${GITHUB_WORKSPACE}/spm" @@ -59,6 +70,7 @@ jobs: addpath(genpath('CanlabCore')); addpath(genpath('Neuroimaging_Pattern_Masks')); addpath(genpath('CANlab_help_examples')); + addpath(genpath('RobustToolbox')); % NB: addpath, not genpath — SPM compat shims would shadow % MATLAB builtins; see test.yml for the same rationale. addpath('spm');