-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Add RigReprojErrorConstantRigCostFunction #2377
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
Add RigReprojErrorConstantRigCostFunction #2377
Conversation
|
I added some benchmarking code. Results:
|
7da99f7 to
a83c8f4
Compare
|
Thanks a lot for the benchmark numbers. I guess the difference is marginal. With the benchmark setup, it's also a little unclear, if the cost function or the constructor leads to the increase, as you only evaluate it once. |
|
I will take another look later but then probably just merge. |
|
Updated the benchmark code to avoid measuring the construction: The numbers are mostly unchanged. |
Bring upstream what we developed for LaMAR.