Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NIDM-Results export #45

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Conversation

cmaumet
Copy link
Member

@cmaumet cmaumet commented Apr 19, 2017

This pull requests implements export of SnPM results as NIDM-Results packs.

@nicholst
Copy link
Collaborator

nicholst commented Apr 19, 2017 via email

@cmaumet cmaumet added the WIP label Apr 19, 2017
@cmaumet
Copy link
Member Author

cmaumet commented Apr 24, 2017

Discussed with @nicholst: computation of Grand Mean Map: can be done as the mean of all inputs around https://github.com/nicholst/SnPM-devel/blob/master/snpm_cp.m#L620-L624, cf. https://github.com/nicholst/SnPM-devel/tree/WriteGrandMean.

@cmaumet cmaumet removed the WIP label Jul 7, 2017
@cmaumet cmaumet changed the title [WIP] NIDM-Results export NIDM-Results export Jul 7, 2017
@cmaumet
Copy link
Member Author

cmaumet commented Jul 7, 2017

  1×70 TestResult array with properties:

    Name
    Passed
    Failed
    Incomplete
    Duration
    Details

Totals:
   70 Passed, 0 Failed, 0 Incomplete.
   438.6952 seconds testing time.

@cmaumet
Copy link
Member Author

cmaumet commented Jul 7, 2017

Next to do:

  • Integrate NIDM export into the GUI (ask user for coordinate space and group info)

@cmaumet
Copy link
Member Author

cmaumet commented Jul 7, 2017

This is now ready to merge. We will wait until the Matlab NIDM-Results generic exporter is released as part of SPM (next release) so that we do not have to ship those files as part of SnPM as well.

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