Skip to content

Cleaning up old code - examples, aa_test, strjoin #160

Description

@jooh

I'm currently working on updating the scripts under examples. The long term goal of this is automated testing, including at least some scripts that can be run outside CBU (finally!).

I have come across a few example scripts that look pretty old. Would it be safe to remove these?

aap_tasklist_begin.xml
aap_tasklist_diffusionFSL.xml
aap_tasklist_end.xml
aap_tasklist_fmri_ppi_fromnifti.xml
aap_tasklist_meg_epochs.xml
aap_tasklist_rsfcMRI.xml
aa_user_diffusionFSL.m
aa_user_macforaws.m
aa_user_meg_aap_testinv2.m
aa_user_meg_aap_test.m
aa_user_meg_aap_test_rhodri.m
aa_user_onaws.m
BIDS114_test1_aa.xml

Relatedly, under aa_testcode there is an old test which seems useful but appears to use a lot of functionality we agreed not to attempt to support at the moment (hyper-alignment). I would like to remove this but even better would be if someone wants to write a new similar test that doesn't require hyper-alignment. In general we need more unit tests (ie tests of individual bits of AA functionality rather than running entire test user scripts).

aatest_dependencies.m

Finally, Matlab includes a strjoin function in recent versions, so it's time for us to get rid of the function by the same name (but slightly different behaviour!) that we ship with AA.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions