View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0003034Slicer4Module N4ITKBiasFieldCorrectionpublic2013-03-27 08:482017-06-10 08:51
Reporterlassoan Assigned Tofedorov  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version 
Target VersionSlicer 4.3.0Fixed in VersionSlicer 4.4.0 
Summary0003034: N4ITKBiasFieldCorrection - tweak accessibly and default of parameters
Description

From Andras:

As I wrote, if another module is available as a replacement then it's fine to retire the MRIBiasFieldCorrection module.

By the way, I think the N4ITK MRI Bias correction module interface reques improvements:

  • The BFWHM parameter - the most important parameter that the most users have to tune - is buried in Advanced N4 parameters / Histogram sharpening parameters / first parameter in the vector.
  • Number of iterations, convergence threshold should be moved to advanced (unlikely to be changed ever)
  • Weight image parameter is unspecified
Tagshelp-wanted

Activities

fedorov

fedorov

2013-03-27 17:58

developer   ~0008257

Items 1 and 2 addressed in r21845.

To Andras:

1) can you add to the documentation the heuristic you use for tuning BFWHM? I have not played with this parameter.

2) what do you mean by "Weight image parameter is unspecified"? The parameter is listed in the XML.
fedorov

fedorov

2013-04-02 09:50

developer   ~0008289

TODO: need to update the baseline images, after the default number of iterations have been updated (num iterations moved into advanced options, and I decided to set larger default number).
fedorov

fedorov

2013-04-08 07:50

developer   ~0008348

baseline updated in r21878
jcfr

jcfr

2014-03-06 05:15

administrator   ~0011062

Closing resolved issues that have not been updated in more than 3 months

Related Changesets

Import 2017-06-07 23:51:09: master ccfee8a5

2014-06-04 14:42:44

jcfr

Details Diff 		ENH: N4ITKBiasFieldCorrection: Restore original default for iteration number.

This commit reverts the change added as part r21845 to resolve issue 0003034

To be compatible with code expecting the original default value, the value
is changed back to to "50,40,30".

From: Andrey Fedorov <fedorov@bwh.harvard.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@23306 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Modules/CLI/N4ITKBiasFieldCorrection/N4ITKBiasFieldCorrection.xml Diff File

Issue History

Date Modified Username Field Change
2013-03-27 08:48 jcfr New Issue
2013-03-27 08:48 jcfr Status new => assigned
2013-03-27 08:48 jcfr Assigned To => fedorov
2013-03-27 08:49 jcfr Reporter jcfr => lassoan
2013-03-27 08:49 jcfr Target Version => Slicer 4.3.0
2013-03-27 08:49 jcfr Tag Attached: I-want-to-contribute
2013-03-27 17:58 fedorov Note Added: 0008257
2013-03-27 17:58 fedorov Status assigned => feedback
2013-04-02 09:50 fedorov Note Added: 0008289
2013-04-02 09:50 fedorov Status feedback => acknowledged
2013-04-08 07:50 fedorov Note Added: 0008348
2013-04-08 09:42 fedorov Status acknowledged => resolved
2013-04-08 09:42 fedorov Resolution open => fixed
2014-03-06 05:15 jcfr Note Added: 0011062
2014-03-06 05:17 jcfr Status resolved => closed
2014-03-06 05:54 jcfr Fixed in Version => Slicer 4.4.0
2014-05-12 23:20 jcfr Tag Renamed I-want-to-contribute => help-wanted
2017-06-10 08:51 jcfr Changeset attached => Slicer master ccfee8a5