-
Notifications
You must be signed in to change notification settings - Fork 235
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
Camera "up" vector is not properly used. #197
Comments
Reopening because #198 broke other usecases and a test, so I reverted it. |
Yes, I can confirm that this fix does not work as intended: Here is an example where the camera is rotated 90° when using the fix, but correctly pointing up with the old version:
|
An attempt to help with google-research#197 google-research#198 Created a new function in Kubric core. set_position() I hope this is helpful xD
An attempt to help with google-research#197 google-research#198 Created a new function in Kubric core. set_position() I hope this is helpful xD
made a new function. It basically just updates the position and then grabs the "look_at" parameter and applies it again. @Qwlouse If you could let me know what other use cases it broke I'll happily attempt to test them xD |
When rendering a scene, everything looks fine with a camera position of
(0., 1.5, -1.5)
. Changing the x-coordinate causes the csene to tilt, and using positive z's renders the scene upside-down.The text was updated successfully, but these errors were encountered: