-
Notifications
You must be signed in to change notification settings - Fork 639
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
What if a partial set of poses is known? #54
Comments
looking at the code, there are preset_pose and preset_focal functions that might do what you want? |
I did overwrite the estimated focal lengths, but since I'm only given a partial set of poses I would like to sort of include it in the optimization problem as a landmark. But thanks for the headsup. |
Internally, we have a slower version of the global alignment where you can force a partial initialisation of camera intrinsics and extrinsics (e.g. focals and poses). |
That would be a very nice feature 😁 |
I'd also be interested in this code; specifically to be able to provide known camera intrinsics to the pipeline. I assume this additional prior should simplify the optimization procedure as well? Thanks for the great work! Very interesting :-) |
I'll take a look. |
I added it in 4a414b6 # here data in list of (intrinsics, pose)
scene = global_aligner(output, device=device, mode=GlobalAlignerMode.ModularPointCloudOptimizer, fx_and_fy=True)
scene.preset_pose([data[i][1].cpu().numpy() for i in range(1, 3)], [False, True, True])
scene.preset_intrinsics([data[i][0].cpu().numpy() for i in range(1, 3)], [False, True, True])
loss = scene.compute_global_alignment(init="mst", niter=niter, schedule=schedule, lr=lr) Note, it won't work if you have only one known pose. |
After trying this I got the following error:loss = scene.compute_global_alignment(init="mst", niter=niter, schedule=schedule, lr=lr)
|
Can this work if I have all the poses known? |
Can you share the whole code which you used to arrive at this error ? |
@gogojjh I also have this issue. Did you make progress? |
@2cu-1001 a possible issue is that the camera intrinsics should be rescaled w.r.t. the image. But I have not tried yet. |
@2cu-1001 you should rescale the camera intrinsics like this
|
@gogojjh thanks a lot! |
Hey Naver,
First of all great work, it is very interesting to play around with!
I'm curious, if one knows a partial set of poses and focal lengths aforehand, how should one initialize the pose-graph?
Best regards
The text was updated successfully, but these errors were encountered: