Proposals:Refactoring Statistics Framework 2007 Miscellaneous: Difference between revisions

From KitwarePublic
Jump to navigationJump to search
Line 7: Line 7:
== CovarianceCalculator ==
== CovarianceCalculator ==
* Known Mean versus estimated mean (incorrect equation: N versus N-1)
* Known Mean versus estimated mean (incorrect equation: N versus N-1)
* http://en.wikipedia.org/wiki/Covariance
== Distance Metric ==
== Distance Metric ==
* Room to expand to other metrics (1_1, 1_inf, Mahalanobis)
* Room to expand to other metrics (1_1, 1_inf, Mahalanobis)

Revision as of 12:25, 3 April 2007

Miscellaneous Issues

Most of these issues were identified by Brad Davis.

CovarianceCalculator

Distance Metric

  • Room to expand to other metrics (1_1, 1_inf, Mahalanobis)

Distance To Centroid Membership Function

  • Why is 1_2 distance hard coded ? as opposed to using a distance metric ?
  • Title centroid is unnecessarily narrow
  • Why not a generic Distance Membership Function where metrics are plugged in ?

Distances

  • itkEuclideanDistance
  • itkMahalanobisDistanceMembershipFunction

ImageToCoocurrenceListAdaptor

  • This is not really an adaptor, it generates new data.
  • Should it be called Generator ? or Filter ?

ListSampleBase

  • Very lightweight, probably unnecessary
  • Only provides a Search() method, and this method probably shouldn't be there

NeighborhoodSampler

  • Sample with spherical (1_2_ neighborhood)
  • Allow other metrics ?

SampleAlgorithmBase

  • Very lightweight, no obvious benefit
  • Is it a candidate for deprecation ?