Ticket #1486 (closed task: fixed)

Opened 8 years ago

Last modified 8 years ago

Reimplement adaptive median code

Reported by: bdezonia Owned by: bdezonia
Priority: major Milestone: imagej2-b5-scripting
Component: Plugins Version:
Severity: serious Keywords:
Cc: Blocked By:
Blocking: #1508

Description (last modified by bdezonia) (diff)

In the NoiseReduction example plugin there was an adaptive median option. When the plugin was broken into multiple separate plugins in the noise reduction menu the adaptive median code was not ported.

This was because when the menu was organized the radial neighborhood code was added. This code relies on a SerialImageAssignment because WithinRadiusOfPointCondition can only work with a single neighborhood. The adaptive median code worked with multiple neighborhoods.

We could make adaptive median available again by avoiding the radial neighborhood implementation. This is not ideal. Otherwise we need to make setAnchor() for all the PointSet code more consistent. There is a separate ticket for that (#1482).

Note that the adaptive median code did not seem to give good results on something like boats. You can still test it by using Cmd L to find the old Noise Reduction plugin and use the adaptive median option. So maybe we don't want to port at all anyhow.

Change History

comment:1 Changed 8 years ago by bdezonia

  • Description modified (diff)

comment:2 Changed 8 years ago by bdezonia

  • Blocking 1460 added

comment:3 Changed 8 years ago by bdezonia

  • Status changed from new to closed
  • Resolution set to fixed

With commit 9746540bb39cd4ea0db5d7479560a79c53214b66 this is now in place. Its operation looks good.

comment:4 Changed 8 years ago by curtis

  • Milestone changed from imagej-2.0.0-beta6 to imagej-2.0.0-beta5-new

comment:5 Changed 8 years ago by bdezonia

  • Blocking 1508 added; 1460 removed
Note: See TracTickets for help on using tickets.