-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[question] visualize sfm_data in ThreeJS #626
Comments
Can it be a columnmajor kind of problem? S. |
I was thinking to the same, can you try to set the transpose of the rotation matrix? |
I would be very interested to see what you come up with @yuyou as I have done a little work in threejs before. :) Please include a link when you can! |
You can get a simplified but working viewer from https://gist.github.com/yuyou/2665238329ca34f0288b073d735a7bcb it uses scripts hosted remotely to save your time for quick testing. Just save the file to your local place from where a file "sfm_data.json" must exist (or change the line 69 accordingly). Optionally, you can place all key frame images under the same directory, and change the variable "load_image" to "true" to show images rendered in the back of the camera "cards". It worked with FireFox and Safari. Chrome works only when the "sfm_data.json" is served from a server over a valid URI address. |
Thank you @yuyou I tested it on some scene and it worked fine. Here a minor comments:
We must see how we can ship it with OpenMVG |
Hi
This is not an issue but could not find other places for helps like this.
I tried to create a simple sfm_data Web viewer by using ThreeJS lib. The original cloud_and_poses PLY file does not show the camera direction (extrinsics). So far I managed to visualize all points and cameras (using THREE.PlaneGeometry) with their position values. My question, however, is how to use the extrinsics' rotation values. I tried to use those values directly as a rotation matrix (4x4), for example, as follows:
The result looks incorrect.
Thanks for any help.
Regards,
Yu
The text was updated successfully, but these errors were encountered: