Revision: 47935
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=47935
Author:   campbellbarton
Date:     2012-06-15 07:50:27 +0000 (Fri, 15 Jun 2012)
Log Message:
-----------
scale node - framing offset: compatible with camera shiftX/Y and the viewport 
option.

Modified Paths:
--------------
    trunk/blender/release/scripts/startup/bl_ui/space_view3d.py
    trunk/blender/source/blender/compositor/nodes/COM_ScaleNode.cpp
    trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.cpp
    trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.h
    trunk/blender/source/blender/editors/space_node/drawnode.c
    trunk/blender/source/blender/makesrna/intern/rna_nodetree.c

Modified: trunk/blender/release/scripts/startup/bl_ui/space_view3d.py
===================================================================
--- trunk/blender/release/scripts/startup/bl_ui/space_view3d.py 2012-06-15 
02:21:07 UTC (rev 47934)
+++ trunk/blender/release/scripts/startup/bl_ui/space_view3d.py 2012-06-15 
07:50:27 UTC (rev 47935)
@@ -2584,14 +2584,14 @@
                         rowsub = col.row()
                         rowsub.prop(bg, "frame_method", expand=True)
 
-                    if bg.view_axis != 'CAMERA':
-                        col.prop(bg, "size")
-
                     row = col.row(align=True)
                     row.prop(bg, "offset_x", text="X")
                     row.prop(bg, "offset_y", text="Y")
 
+                    if bg.view_axis != 'CAMERA':
+                        col.prop(bg, "size")
 
+
 class VIEW3D_PT_transform_orientations(Panel):
     bl_space_type = 'VIEW_3D'
     bl_region_type = 'UI'

Modified: trunk/blender/source/blender/compositor/nodes/COM_ScaleNode.cpp
===================================================================
--- trunk/blender/source/blender/compositor/nodes/COM_ScaleNode.cpp     
2012-06-15 02:21:07 UTC (rev 47934)
+++ trunk/blender/source/blender/compositor/nodes/COM_ScaleNode.cpp     
2012-06-15 07:50:27 UTC (rev 47935)
@@ -65,8 +65,12 @@
                case CMP_SCALE_RENDERPERCENT: {
                        const RenderData *data = &context->getScene()->r;
                        ScaleFixedSizeOperation *operation = new 
ScaleFixedSizeOperation();
+
+                       /* framing options */
                        operation->setIsAspect((bnode->custom2 & 
CMP_SCALE_RENDERSIZE_FRAME_ASPECT) != 0);
                        operation->setIsCrop((bnode->custom2 & 
CMP_SCALE_RENDERSIZE_FRAME_CROP) != 0);
+                       operation->setOffset(bnode->custom3, bnode->custom4);
+
                        operation->setNewWidth(data->xsch * data->size / 
100.0f);
                        operation->setNewHeight(data->ysch * data->size / 
100.0f);
                        
inputSocket->relinkConnections(operation->getInputSocket(0), 0, graph);

Modified: 
trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.cpp
===================================================================
--- trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.cpp   
2012-06-15 02:21:07 UTC (rev 47934)
+++ trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.cpp   
2012-06-15 07:50:27 UTC (rev 47935)
@@ -191,6 +191,21 @@
        this->relX = inputOperation->getWidth() / (float)this->newWidth;
        this->relY = inputOperation->getHeight() / (float)this->newHeight;
 
+
+       /* *** all the options below are for a fairly special case - camera 
framing *** */
+       if (this->offsetX != 0.0f || this->offsetY != 0.0f) {
+               this->is_offset = true;
+
+               if (this->newWidth > this->newHeight) {
+                       this->offsetX *= this->newWidth;
+                       this->offsetY *= this->newWidth;
+               }
+               else {
+                       this->offsetX *= this->newHeight;
+                       this->offsetY *= this->newHeight;
+               }
+       }
+
        if (this->is_aspect) {
                /* apply aspect from clip */
                const float w_src = inputOperation->getWidth();
@@ -203,9 +218,6 @@
                const float asp_src = w_src / h_src;
                const float asp_dst = w_dst / h_dst;
 
-               this->offsetX = 0.0f;
-               this->offsetY = 0.0f;
-
                if (fabsf(asp_src - asp_dst) >= FLT_EPSILON) {
                        if ((asp_src > asp_dst) == (this->is_crop == true)) {
                                /* fit X */
@@ -219,9 +231,11 @@
                                this->relY /= div;
                                this->offsetY = ((h_src - (h_src * div)) / 
(h_src / h_dst)) / 2.0f;
                        }
+
+                       this->is_offset = true;
                }
-
        }
+       /* *** end framing options *** */
 }
 
 void ScaleFixedSizeOperation::deinitExecution()
@@ -236,7 +250,7 @@
        sampler = COM_PS_BICUBIC;
 #endif
 
-       if (this->is_aspect) {
+       if (this->is_offset) {
                float nx = ((x - this->offsetX) * relX);
                float ny = ((y - this->offsetY) * relY);
                this->inputOperation->read(color, nx, ny, sampler, 
inputBuffers);

Modified: 
trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.h
===================================================================
--- trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.h     
2012-06-15 02:21:07 UTC (rev 47934)
+++ trunk/blender/source/blender/compositor/operations/COM_ScaleOperation.h     
2012-06-15 07:50:27 UTC (rev 47935)
@@ -69,6 +69,9 @@
        float offsetY;
        bool is_aspect;
        bool is_crop;
+       /* set from other properties on initialization,
+        * check if we need to apply offset */
+       bool is_offset;
 public:
        ScaleFixedSizeOperation();
        bool determineDependingAreaOfInterest(rcti *input, ReadBufferOperation 
*readOperation, rcti *output);
@@ -79,8 +82,9 @@
        void deinitExecution();
        void setNewWidth(int width) { this->newWidth = width; }
        void setNewHeight(int height) { this->newHeight = height; }
-       void setIsAspect(int is_aspect) { this->is_aspect = is_aspect; }
-       void setIsCrop(int is_crop) { this->is_crop = is_crop; }
+       void setIsAspect(bool is_aspect) { this->is_aspect = is_aspect; }
+       void setIsCrop(bool is_crop) { this->is_crop = is_crop; }
+       void setOffset(float x, float y) { this->offsetX = x; this->offsetY = 
y; }
 };
 
 #endif

Modified: trunk/blender/source/blender/editors/space_node/drawnode.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/drawnode.c  2012-06-15 
02:21:07 UTC (rev 47934)
+++ trunk/blender/source/blender/editors/space_node/drawnode.c  2012-06-15 
07:50:27 UTC (rev 47935)
@@ -2020,7 +2020,11 @@
        uiItemR(layout, ptr, "space", 0, "", ICON_NONE);
 
        if (RNA_enum_get(ptr, "space") == CMP_SCALE_RENDERPERCENT) {
+               uiLayout *row;
                uiItemR(layout, ptr, "frame_method", UI_ITEM_R_EXPAND, NULL, 
ICON_NONE);
+               row = uiLayoutRow(layout, TRUE);
+               uiItemR(row, ptr, "offset_x", 0, "X", ICON_NONE);
+               uiItemR(row, ptr, "offset_y", 0, "Y", ICON_NONE);
        }
 }
 

Modified: trunk/blender/source/blender/makesrna/intern/rna_nodetree.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/rna_nodetree.c 2012-06-15 
02:21:07 UTC (rev 47934)
+++ trunk/blender/source/blender/makesrna/intern/rna_nodetree.c 2012-06-15 
07:50:27 UTC (rev 47935)
@@ -2106,6 +2106,16 @@
        RNA_def_property_enum_items(prop, space_frame_items);
        RNA_def_property_ui_text(prop, "Frame Method", "How the image fits in 
the camera frame");
        RNA_def_property_update(prop, NC_NODE | NA_EDITED, "rna_Node_update");
+
+       prop = RNA_def_property(srna, "offset_x", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "custom3");
+       RNA_def_property_ui_text(prop, "X Offset", "Offset image horizontally 
(factor of image size)");
+       RNA_def_property_update(prop, NC_NODE | NA_EDITED, "rna_Node_update");
+
+       prop = RNA_def_property(srna, "offset_y", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "custom4");
+       RNA_def_property_ui_text(prop, "Y Offset", "Offset image vertically 
(factor of image size)");
+       RNA_def_property_update(prop, NC_NODE | NA_EDITED, "rna_Node_update");
 }
 
 static void def_cmp_rotate(StructRNA *srna)

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to