diff options
author | Dalai Felinto <dfelinto@gmail.com> | 2015-06-29 16:24:25 +0300 |
---|---|---|
committer | Dalai Felinto <dfelinto@gmail.com> | 2015-06-29 16:24:25 +0300 |
commit | 3d12d4b94f9bff6ea810179e22a5e3e018541f31 (patch) | |
tree | b7a0e7d2bc49c0d5757c8a8394b60ff99a9a2900 | |
parent | 295d0c52a26730edc6d4ed1276e4051cce006be5 (diff) |
Fix T45234: Stereo Parallel vs. Off-Axis
Parallel rendering was not working.
The idea of having parallel convergence mode to render as parallel but
visualize as off-axis was good, but it was leading to some complications
in the code.
I think it's more clear to the user if parallel looks and render as
parallel, and if she wants to pre-visualize the converged planes, simply
temporarily set the camera to off-axis.
-rw-r--r-- | release/scripts/startup/bl_ui/properties_data_camera.py | 7 | ||||
-rw-r--r-- | source/blender/blenkernel/intern/camera.c | 9 | ||||
-rw-r--r-- | source/blender/makesrna/intern/rna_camera.c | 9 |
3 files changed, 7 insertions, 18 deletions
diff --git a/release/scripts/startup/bl_ui/properties_data_camera.py b/release/scripts/startup/bl_ui/properties_data_camera.py index 24a67a22c1f..92ec34e01b2 100644 --- a/release/scripts/startup/bl_ui/properties_data_camera.py +++ b/release/scripts/startup/bl_ui/properties_data_camera.py @@ -153,10 +153,9 @@ class DATA_PT_camera_stereoscopy(CameraButtonsPanel, Panel): col = layout.column() col.row().prop(st, "convergence_mode", expand=True) - if st.convergence_mode == 'PARALLEL': - col.prop(st, "viewport_convergence") - else: - col.prop(st, "convergence_distance") + sub = col.column() + sub.active = st.convergence_mode == 'OFFAXIS' + sub.prop(st, "convergence_distance") col.prop(st, "interocular_distance") diff --git a/source/blender/blenkernel/intern/camera.c b/source/blender/blenkernel/intern/camera.c index 87d81a9e921..b308dc7a262 100644 --- a/source/blender/blenkernel/intern/camera.c +++ b/source/blender/blenkernel/intern/camera.c @@ -908,6 +908,9 @@ static float camera_stereo3d_shift_x(Object *camera, const char *viewname) convergence_mode = data->stereo.convergence_mode; pivot = data->stereo.pivot; + if (convergence_mode != CAM_S3D_OFFAXIS) + return shift; + if (((pivot == CAM_S3D_PIVOT_LEFT) && is_left) || ((pivot == CAM_S3D_PIVOT_RIGHT) && !is_left)) { @@ -918,11 +921,7 @@ static float camera_stereo3d_shift_x(Object *camera, const char *viewname) fac = 0.5f; fac_signed = is_left ? fac : -fac; - - /* Note: in viewport, parallel renders as offaxis, but in render it does parallel */ - if (ELEM(convergence_mode, CAM_S3D_OFFAXIS, CAM_S3D_PARALLEL)) { - shift += ((interocular_distance / data->sensor_x) * (data->lens / convergence_distance)) * fac_signed; - } + shift += ((interocular_distance / data->sensor_x) * (data->lens / convergence_distance)) * fac_signed; return shift; } diff --git a/source/blender/makesrna/intern/rna_camera.c b/source/blender/makesrna/intern/rna_camera.c index 16f74a05f7b..9aec0ea43cc 100644 --- a/source/blender/makesrna/intern/rna_camera.c +++ b/source/blender/makesrna/intern/rna_camera.c @@ -144,15 +144,6 @@ static void rna_def_camera_stereo_data(BlenderRNA *brna) "The converge point for the stereo cameras " "(often the distance between a projector and the projection screen)"); RNA_def_property_update(prop, NC_OBJECT | ND_DRAW, NULL); - - prop = RNA_def_property(srna, "viewport_convergence", PROP_FLOAT, PROP_DISTANCE); - RNA_def_property_float_sdna(prop, NULL, "convergence_distance"); - RNA_def_property_range(prop, 0.00001f, FLT_MAX); - RNA_def_property_ui_range(prop, 0.0f, 15.f, 1, 2); - RNA_def_property_ui_text(prop, "Viewport Convergence", - "Preview convergence distance for the stereo effect in the viewport " - "(it does not affect the render!)"); - RNA_def_property_update(prop, NC_OBJECT | ND_DRAW, NULL); } void RNA_def_camera(BlenderRNA *brna) |