imagej / imagej-ops

ImageJ Ops: "Write once, run anywhere" image processing
https://imagej.net/libs/imagej-ops
BSD 2-Clause "Simplified" License
88 stars 42 forks source link

Improve FeatureSet Tests #449

Open tibuch opened 7 years ago

tibuch commented 7 years ago

Test and improve FeatureSets.

tibuch commented 7 years ago

@ctrueden and @dietzc why do we have expensive geom tests and normal ones?

dietzc commented 7 years ago

I think for historical reasons. If you find "normal" ones which cover all the cases, it's absolutely OK.

ctrueden commented 7 years ago

@tibuch I thought you were the one who authored those expensive tests! No?

I would much rather have only fast-running tests but which still exercise the algorithms sufficiently. But my understanding was that someone (you? someone else?) thought the small ones might not be rich/complex enough to fully exercise the algorithms?

tibuch commented 7 years ago

@ctrueden I guess the tests were added by @seebacherd.

I will replace the images by smaller but complexer (curves, holes and so on) ones.

I would say the accuracy of the tests depends a lot more on small epsilons.

ctrueden commented 7 years ago

@tibuch Thanks, sounds great!