Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ellipsoid #28

Closed
tibuch opened this issue Oct 6, 2016 · 2 comments
Closed

Ellipsoid #28

tibuch opened this issue Oct 6, 2016 · 2 comments

Comments

@tibuch
Copy link

tibuch commented Oct 6, 2016

I am improving the geometric 2d feature tests in imagej-ops imagej/imagej-ops#449. Since I couldn't verify the major- and minor-axis numbers I took a closer look at the implementation. Currently ImageJ1 code which is adapted to work with a Polygon is used to find the fitting ellipse from which major- and minor-axis are extracted.

Since I couldn't verify the numbers I opened this discussion imagej/imagej-ops#452 which ended up in this PR imagej/imagej-ops#453 which would decouple the fitting ellipse implementation from the major- and minor-axis implementation and could be tested against a matlab-implementation.

This is not the best solution, because this implementation is based on the deprecated ellipse implementation. In discussion with @tpietzsch and @dietzc I found other implementations of HyperEllipsoids and fitting ellipse algorithms.

What do we have:

What I would suggest:

  • An ellipse implementation on imglib2 side.
  • A fitting ellipse implementation on imglib2 side or on imagej-ops side.

I am happy to contribute, the question is just where to continue.

@maierl
Copy link

maierl commented Oct 13, 2016

There's another 3D Ellipsoid implementation available as ImageJ plugin: https://github.com/mcib3d/mcib3d-plugins/blob/master/src/main/java/mcib_plugins/Ellipsoids_3D.java

@tibuch
Copy link
Author

tibuch commented Nov 23, 2016

I was able to verify the major-minor-axis implementation.

The best fitting ellipse in the major-minor-axis case, is the one with the "same" area. which is different from the referenced implementations which fit an ellipse to a point cloud.

Also ellipse is part of this discussion: http://forum.imagej.net/t/implementation-plan-for-imglib2-rois-2d/2531/1

@tibuch tibuch closed this as completed Nov 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants