Ticket #1486 (closed task: fixed)
Opened 2012-09-17T15:46:42-05:00
Last modified 2012-10-10T15:18:52-05:00
Reimplement adaptive median code
Reported by: | bdezonia | Owned by: | bdezonia |
---|---|---|---|
Priority: | major | Milestone: |
|
Component: | Plugins | Version: | |
Severity: | serious | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | #1508 |
Description (last modified by bdezonia)
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:3 Changed 2012-10-09T14:14:33-05:00 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 2012-10-10T13:27:49-05:00 by curtis
- Milestone changed from imagej-2.0.0-beta6 to imagej-2.0.0-beta5-new