Skip to content

Tibor - #149

Merged
jooh merged 64 commits into
automaticanalysis:masterfrom
tiborauer:master
Jun 25, 2018
Merged

Tibor#149
jooh merged 64 commits into
automaticanalysis:masterfrom
tiborauer:master

Conversation

@tiborauer

Copy link
Copy Markdown
Member

tiborauer and others added 24 commits March 30, 2018 18:52
UPDATE: qsub_nonDCS: Worker information
@jooh

jooh commented May 3, 2018

Copy link
Copy Markdown
Collaborator

Running tests now...

@jooh

jooh commented May 3, 2018

Copy link
Copy Markdown
Collaborator

I'm getting a crash in aa_modules/aamod_convert_specialseries.m while running through the ASL pipeline. The line below is problematic, because d is a cell array with one entry, not an integer or boolean (ie, can't be used to index into dcmhdr).

spm_file_merge(char(img4D),fname4D,0,dcmhdr{d}.volumeTR);

@tiborauer

Copy link
Copy Markdown
Member Author

I have updated the PR. How about now?

@jooh

jooh commented May 29, 2018

Copy link
Copy Markdown
Collaborator

Hi Tibor,

Still can't get aa_user_MTI to run I'm afraid:

Job40 on <a href="matlab: cd('/imaging/jc01/aatest/cbu/tmp/MTI_aa/MTI/aamod_dartel_norm_write_00001/S01')">subject "S01" - MRI:CBU140905_CAMCAN_CALIBRATIONS/20141001_153018</a> had an error: aa error:
No inputs obtained for stream structural! 
        Module aamod_segment8_multichan might not have crteated it.

@tiborauer

Copy link
Copy Markdown
Member Author

I have updated the PR!

@jooh

jooh commented Jun 11, 2018

Copy link
Copy Markdown
Collaborator

Ok, running the tests again. This could be the one that passes!

@jooh

jooh commented Jun 12, 2018

Copy link
Copy Markdown
Collaborator

We still crash on AA MTI. Can you make sure aa_user_MTI runs before updating the PR?

=============================================================
Parallel process ID 97540
INFO: Current MATLAB version 9.2.0.556344 (R2017a) suitable for this aa 5.4.0 (cb0cf0b6ff55a35af77465208ac959cc85d43285) Jun 2018 

INFO: Current aa version 5.4.0 (cb0cf0b6ff55a35af77465208ac959cc85d43285) Jun 2018 suitable for this user script

INITIALISATION MODULE aamod_checkparameters: Check user parameters
INITIALISATION MODULE aamod_evaluatesubjectnames: Check raw data for subject "S01" - MRI:140905 
INITIALISATION MODULE aamod_evaluatesubjectnames: Check raw data for subject "S02" - MRI:140910
INITIALISATION MODULE aamod_study_init: aa initialise study
INITIALISATION MODULE aamod_newsubj_init: Create subject directory for subject "S01" - MRI:CBU140905_CAMCAN_CALIBRATIONS/20141001_153018
INITIALISATION MODULE aamod_newsubj_init: Create subject directory for subject "S02" - MRI:CBU140910_CAMCAN_CALIBRATIONS/20141002_155503
Stage aamod_get_dicom_structural input autoidentifyseries comes from aamod_autoidentifyseries_timtrio which is 1 dependencies prior
Stage aamod_convert_structural input dicom_structural comes from aamod_get_dicom_structural which is 1 dependencies prior
Stage aamod_biascorrect_structural input structural comes from aamod_convert_structural which is 1 dependencies prior 
Stage aamod_coreg_extended_1 input structural comes from aamod_biascorrect_structural which is 1 dependencies prior
Stage aamod_segment8_multichan input structural comes from aamod_coreg_extended_1 which is 1 dependencies prior 
Stage aamod_dartel_createtemplate input dartelimported_grey comes from aamod_segment8_multichan which is 1 dependencies prior 
Stage aamod_dartel_createtemplate input dartelimported_white comes from aamod_segment8_multichan which is 1 dependencies prior
Stage aamod_dartel_norm_write input structural comes from aamod_segment8_multichan which is 2 dependencies prior
Stage aamod_dartel_norm_write input dartel_template comes from aamod_dartel_createtemplate which is 1 dependencies prior
Stage aamod_dartel_norm_write input dartel_flowfield comes from aamod_dartel_createtemplate which is 1 dependencies prior
Stage aamod_dartel_norm_write input dartel_templatetomni_xfm comes from aamod_dartel_createtemplate which is 1 dependencies prior
Stage aamod_get_dicom_specialseries input autoidentifyseries comes from aamod_autoidentifyseries_timtrio which is 8 dependencies prior
Stage aamod_convert_specialseries input dicom_MTI comes from aamod_get_dicom_specialseries which is 1 dependencies prior
Stage aamod_MTI2MTR input MTI comes from aamod_convert_specialseries which is 1 dependencies prior
Stage aamod_MTI2MTR input MTI_dicom_header comes from aamod_convert_specialseries which is 1 dependencies prior
Stage aamod_MTI_coreg_extended_2 input t1totemplate_xfm comes from aamod_coreg_extended_1 which is 7 dependencies prior
Stage aamod_MTI_coreg_extended_2 input MTR comes from aamod_MTI2MTR which is 1 dependencies prior
Stage aamod_MTI_coreg_extended_2 input MTI_baseline comes from aamod_MTI2MTR which is 1 dependencies prior
Stage aamod_MTI_coreg_extended_2 input aamod_coreg_extended_1_00001.structural comes from aamod_coreg_extended_1 which is 7 dependencies prior
Stage aamod_MTI_norm_write_dartel input dartel_template comes from aamod_dartel_createtemplate which is 6 dependencies prior
Stage aamod_MTI_norm_write_dartel input dartel_templatetomni_xfm comes from aamod_dartel_norm_write which is 5 dependencies prior
Stage aamod_MTI_norm_write_dartel input dartel_flowfield comes from aamod_dartel_createtemplate which is 6 dependencies prior
Stage aamod_MTI_norm_write_dartel input structural comes from aamod_dartel_norm_write which is 5 dependencies prior
Stage aamod_MTI_norm_write_dartel input MTR comes from aamod_MTI_coreg_extended_2 which is 1 dependencies prior
Stage aamod_MTI_roi_extract input MTR comes from aamod_MTI_norm_write_dartel which is 1 dependencies prior
Stage aamod_MTI_roi_valid input roidata_MTR comes from aamod_MTI_roi_extract which is 1 dependencies prior
Stage aamod_MTI_roi_extract input MTR comes from aamod_MTI_norm_write_dartel which is 1 dependencies prior
Stage aamod_MTI_roi_valid input roidata_MTR comes from aamod_MTI_roi_extract which is 1 dependencies prior
/usr/bin/dot
INFO: pool profile CBU_Cluster found
INFO: pool Torque is detected
WARNING: Renaming output structural of aamod_segment8_multichan_00001 to .
  It might breake dependency for the consecuting module(s).
  Make sure that you know what you are doing!
REMOVED: aamod_segment8_multichan_00001 output stream: structural
WARNING: Renaming output t2 of aamod_segment8_multichan_00001 to .
  It might breake dependency for the consecuting module(s).
  Make sure that you know what you are doing!
REMOVED: aamod_segment8_multichan_00001 output stream: t2
WARNING: Renaming output structural of aamod_segment8_multichan_00001 to .
  It might breake dependency for the consecuting module(s).
  Make sure that you know what you are doing!


**** automatic analysis failed - see reason and line numbers below
ERROR: Stream structural of module aamod_segment8_multichan not found!

tiborauer and others added 2 commits June 13, 2018 18:21
@tiborauer

Copy link
Copy Markdown
Member Author

I have updated the PR and tested aa_user_MTI

@jooh

jooh commented Jun 18, 2018

Copy link
Copy Markdown
Collaborator

We made it past MTI, yay! But now we have a crash in aa report during aa_user_diffusion_with_topup:

ans =
    'aa error:
     Inputstream dicom_diffusion of module aamod_convert_diffusion_phaseencode_direction_00001 generated by aamod_get_dicom_diffusion_00001 not found!
     '
K>> err.stack(1)
ans = 
  struct with fields:

    file: '/home/jc01/matlab/aa5/aa_engine/aas_log.m'
    name: 'aas_log'
    line: 63
K>> err.stack(2)
ans = 
  struct with fields:

    file: '/home/jc01/matlab/aa5/aa_toolbox/provenance/aa_provenance.m'
    name: 'aa_provenance.addModule'
    line: 243
K>> err.stack(3)
ans = 
  struct with fields:

    file: '/home/jc01/matlab/aa5/aa_toolbox/aa_report.m'
    name: 'aa_report'
    line: 94
K>> err.stack(4)
ans = 
  struct with fields:

    file: '/imaging/jc01/aatest/cbu/aa_user_diffusion_with_topup.m'
    name: 'aa_user_diffusion_with_topup'
    line: 60

@tiborauer

Copy link
Copy Markdown
Member Author

The reporting was not prepared for subsessions (i.e. AP-PA). I have updated the PR.

@jooh

jooh commented Jun 20, 2018

Copy link
Copy Markdown
Collaborator

Ok, running the tests again...

@jooh
jooh merged commit 2ed6c78 into automaticanalysis:master Jun 25, 2018
tiborauer added a commit to INCF/automaticanalysis that referenced this pull request Jul 8, 2021
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.

3 participants