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

Render VR180 video #2261

Merged
merged 2 commits into from
Jul 24, 2023
Merged

Render VR180 video #2261

merged 2 commits into from
Jul 24, 2023

Conversation

cvachha
Copy link
Contributor

@cvachha cvachha commented Jul 23, 2023

Adding the VR180 camera model to render VR180 video and images. I condensed the omnidirectional stereo and VR180 rendering post processing in the render script. I have updated the documentation for VR180 support as well as the Blender documentation since VR180 rendering is compatible with the Blender add-on.

Adding the VR180 camera model to render VR180 video and images. I condensed the omnidirectional stereo and VR180 render post processing in the render script. I have updated the documentation for VR180 support.
Comment on lines 157 to 161
camera_type == CameraType.EQUIRECTANGULAR
or camera_type == CameraType.OMNIDIRECTIONALSTEREO_L
or camera_type == CameraType.OMNIDIRECTIONALSTEREO_R
or camera_type == CameraType.VR180_L
or camera_type == CameraType.VR180_R
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you condense this to
`if camera_type in [CameraType.EQUIRECTANGULAR, ...]

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good, I have updated the formatting.

Condensing the if statement for camera type intrinsics.
Copy link
Contributor

@tancik tancik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@tancik tancik merged commit 8682e48 into main Jul 24, 2023
4 checks passed
@tancik tancik deleted the cyrus/vr180_rendering branch July 24, 2023 14:26
@Ilyabasharov
Copy link
Contributor

Hello, @cvachha! Could you please fix bug here. What did you mean?

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.

None yet

3 participants