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

Orient scale #1613

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Orient scale #1613

wants to merge 2 commits into from

Conversation

drgallaci
Copy link

Existing Orient method (solve - overdetermined) provided sub optimal reorientation, furthermore only rotation and translation are implemented, no options for x/y scaling.
#700 (geometric error compensation)
I've created a new method for the task. It's a primitive and naive evolutionary-like algorithm to get more optimal rotation, translation and scaling parameters for Orient. Not too sophisticated but provides great results using only 250-400 iterations (well_subsecond).
I got more precise positions than using the old solver method (even by the time I have not implemented scaling yet).
I needed scaling badly, so now I get great drill positions after Orient.

[Changing Orient stuff (e.g. active marker slider) reexecutes the algorithm (just like it did before), 400 more iterations are executed, thus slightly improves parameters and gets lower errors. Rather a feature than bug.]

image

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

Successfully merging this pull request may close these issues.

1 participant