Show errors for missing images in reports. about fmriprep HOT 7 CLOSED

I would suggest a different approach. After running the workflow that exits with errors

• crawl the crashfile directory and identify all crashfiles belonging to each subject analyzed in this run (you might need additional daytime restrictions to avoid capturing crashfiles from historical runs)
• append error information to corresponding subject specific HTML reports report

This way we will be able to capture all errors (not only those coming from Report Capable interfaces). It will also avoid having to deal with the problematic case of a failed node that populates error reportlet that should be send to a datasink (what @oesteban mentioned in #66 (comment)).

from fmriprep.

Actually, the problem of confusing crash files from different subjects and historical runs could be solved, by using a unique temporary crashfile directory for every subject/run.

from fmriprep.

I think this is currently well covered with the new reports.

from fmriprep.

Yep, the problem is that the out_report output trait is never populated: the node fails and, even though a reportlet is generated, the nipype engine will not consider it.

from fmriprep.

The reports should include the content of all generated crashfiles. Here's the code for reading crashfiles: https://github.com/nipy/nipype/blob/master/nipype/scripts/crash_files.py (could be easily adapted with mocking print).

We briefly discussed adding optional plain text crashfiles to nipype (see nipy/nipype#1184), but I doubt this can happen before the next nipype release (rc1 was just released).

from fmriprep.

In niworkflows for the definition of ReportCapableInterface there is check to see if the interface ran successfully and if not to call _generate_error_report: https://github.com/poldracklab/niworkflows/blob/master/niworkflows/common/report.py#L59

I'm looking into writing the crash information out to the reportlet here. That being said in the reports that we have on afterhours I'm not seeing any instances of the html from _generate_error_report.

from fmriprep.

Closed by #298

from fmriprep.