View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002273 | Slicer4 | Core: CLI infrastructure | public | 2012-06-28 05:35 | 2014-09-17 23:01 |
Reporter | DMeier | Assigned To | lassoan | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | Slicer 4.1.1 | ||||
Target Version | Slicer 4.4.0 | Fixed in Version | Slicer 4.4.0 | ||
Summary | 0002273: Demons Registration (BRAINS) does not produce deformation field / cannot save | ||||
Description | Demons Registration (BRAINS) module has as output option an "Output Displacement Field Volume". What is generated looks like a .tfm node instead, when a volume would be expected. Slicer will not allow saving of this transform and reports: "Cannot write data file ...Demons.tfm" | ||||
Tags | No tags attached. | ||||
<transform fileExtensions=".nrrd" type="nonlinear" reference="movingVolume"> ================ There are several hacks that we could put into place, but I don't know which one is the least distastful: DMeier, would you be able to test that? Hans |
|
This might have been fixed by recent nonlinear transform improvements. @Dominik can you doublecheck? |
|
@Alex please look into this and close the issue if it's resolved. |
|
Andras, saving GridTransform node causes crash now (both as .nrrd or .tfm) |
|
I confirm that latest nightly version Slicer crashes with the if you attempt to save an uninitialized grid transform (a node is created but not a displacement field image is not set). If you actually perform a demons registration and a field is computed then saving/loading of that field works well. The fix for the crash (it logs a meaningful error message instead of crashing) is already available but not yet merged: |
|
Alex can you look at the pull request? If it looks good you can merge it to svn using the instructions here: |
|
The note on the crash: The CLI completes and I don't see errors reported. However Slicer crashes on save with null DeformationField (which I understand Andras has fixed). Still I would expect to see an error reported from CLI if it cannot create a valid Grid transform. I don't know if the problem is in Demons CLI or slicer. Having said this, the code looks reasonable, so I suggest Andras merges it into trunk. |
|
Resolution merged into trunk: |
|
Closing resolved issues that have not been updated in more than 3 months. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2012-06-28 05:35 | DMeier | New Issue | |
2012-08-20 08:20 | nicole | Assigned To | => hjmjohnson |
2012-08-20 08:20 | nicole | Status | new => assigned |
2012-08-20 08:20 | nicole | Target Version | => Slicer 4.2.0 - Feature freeze Sept 1st 2012 |
2012-08-20 12:31 | jcfr | Target Version | Slicer 4.2.0 - Feature freeze Sept 1st 2012 => Slicer 4.2.5 |
2012-08-21 09:40 | jcfr | Target Version | Slicer 4.2.5 => Slicer 4.3.0 |
2013-07-05 07:19 | hjmjohnson | Note Added: 0008863 | |
2013-08-27 12:31 | jcfr | Assigned To | hjmjohnson => alexy |
2013-08-27 12:32 | pieper | Target Version | Slicer 4.3.0 => Slicer 4.4.0 |
2014-03-07 06:43 | pieper | Note Added: 0011337 | |
2014-04-15 12:48 | pieper | Note Added: 0011592 | |
2014-05-16 07:38 | alexy | Note Added: 0011869 | |
2014-05-16 07:38 | alexy | Assigned To | alexy => lassoan |
2014-05-16 10:30 | lassoan | Note Added: 0011870 | |
2014-05-16 10:31 | lassoan | Note Edited: 0011870 | |
2014-05-16 10:57 | pieper | Note Added: 0011871 | |
2014-05-16 11:18 | alexy | Note Added: 0011872 | |
2014-05-16 13:21 | lassoan | Note Added: 0011878 | |
2014-05-16 13:21 | lassoan | Status | assigned => resolved |
2014-05-16 13:21 | lassoan | Fixed in Version | => Slicer 4.4.0 |
2014-05-16 13:21 | lassoan | Resolution | open => fixed |
2014-09-17 22:59 | jcfr | Status | resolved => closed |
2014-09-17 23:01 | jcfr | Note Added: 0012538 |