Commit: b20449cc08f33fc6b55fc6c326f5ab1b5f240fb9
Author: Sergey Sharybin
Date:   Wed May 23 12:13:21 2018 +0200
Branches: blender2.8
https://developer.blender.org/rBb20449cc08f33fc6b55fc6c326f5ab1b5f240fb9

Cycles/Render API: changes for better copy-on-write support.

Mainly just passing the depsgraph and evaluated scene and camera, instead of
the original one. Patch by Sergey, further modifications by Brecht.

===================================================================

M       intern/cycles/blender/addon/__init__.py
M       intern/cycles/blender/addon/engine.py
M       intern/cycles/blender/blender_python.cpp
M       intern/cycles/blender/blender_session.cpp
M       intern/cycles/blender/blender_session.h
M       intern/cycles/blender/blender_sync.cpp
M       source/blender/editors/object/object_bake_api.c
M       source/blender/makesrna/intern/rna_render.c
M       source/blender/render/extern/include/RE_bake.h
M       source/blender/render/extern/include/RE_engine.h
M       source/blender/render/intern/source/external_engine.c

===================================================================

diff --git a/intern/cycles/blender/addon/__init__.py 
b/intern/cycles/blender/addon/__init__.py
index 9bf9a3839d2..353f2a1ad5b 100644
--- a/intern/cycles/blender/addon/__init__.py
+++ b/intern/cycles/blender/addon/__init__.py
@@ -66,31 +66,32 @@ class CyclesRender(bpy.types.RenderEngine):
         engine.free(self)
 
     # final render
-    def update(self, data, scene):
+    def update(self, data, depsgraph):
         if not self.session:
             if self.is_preview:
                 cscene = bpy.context.scene.cycles
                 use_osl = cscene.shading_system and cscene.device == 'CPU'
 
-                engine.create(self, data, scene,
-                              None, None, None, use_osl)
+                engine.create(self, data, preview_osl=use_osl)
             else:
-                engine.create(self, data, scene)
-        else:
-            engine.reset(self, data, scene)
+                engine.create(self, data)
+
+        engine.reset(self, data, depsgraph)
 
     def render_to_image(self, depsgraph):
         engine.render(self, depsgraph)
 
-    def bake(self, depsgraph, scene, obj, pass_type, pass_filter, object_id, 
pixel_array, num_pixels, depth, result):
+    def bake(self, depsgraph, obj, pass_type, pass_filter, object_id, 
pixel_array, num_pixels, depth, result):
         engine.bake(self, depsgraph, obj, pass_type, pass_filter, object_id, 
pixel_array, num_pixels, depth, result)
 
     # viewport render
     def view_update(self, context):
         if not self.session:
-            engine.create(self, context.blend_data, context.scene,
+            engine.create(self, context.blend_data,
                           context.region, context.space_data, 
context.region_data)
-        engine.update(self, context.depsgraph, context.blend_data, 
context.scene)
+
+        engine.reset(self, context.blend_data, context.depsgraph)
+        engine.sync(self, context.depsgraph, context.blend_data)
 
     def render_to_view(self, context):
         engine.draw(self, context.depsgraph, context.region, 
context.space_data, context.region_data)
diff --git a/intern/cycles/blender/addon/engine.py 
b/intern/cycles/blender/addon/engine.py
index 8a13b67f91d..b689c750be0 100644
--- a/intern/cycles/blender/addon/engine.py
+++ b/intern/cycles/blender/addon/engine.py
@@ -123,13 +123,12 @@ def exit():
     _cycles.exit()
 
 
-def create(engine, data, scene, region=None, v3d=None, rv3d=None, 
preview_osl=False):
-    import bpy
+def create(engine, data, region=None, v3d=None, rv3d=None, preview_osl=False):
     import _cycles
+    import bpy
 
     data = data.as_pointer()
     userpref = bpy.context.user_preferences.as_pointer()
-    scene = scene.as_pointer()
     if region:
         region = region.as_pointer()
     if v3d:
@@ -137,13 +136,8 @@ def create(engine, data, scene, region=None, v3d=None, 
rv3d=None, preview_osl=Fa
     if rv3d:
         rv3d = rv3d.as_pointer()
 
-    if bpy.app.debug_value == 256:
-        _cycles.debug_flags_update(scene)
-    else:
-        _cycles.debug_flags_reset()
-
     engine.session = _cycles.create(
-            engine.as_pointer(), userpref, data, scene, region, v3d, rv3d, 
preview_osl)
+            engine.as_pointer(), userpref, data, region, v3d, rv3d, 
preview_osl)
 
 
 def free(engine):
@@ -167,14 +161,21 @@ def bake(engine, depsgraph, obj, pass_type, pass_filter, 
object_id, pixel_array,
         _cycles.bake(engine.session, depsgraph.as_pointer(), obj.as_pointer(), 
pass_type, pass_filter, object_id, pixel_array.as_pointer(), num_pixels, depth, 
result.as_pointer())
 
 
-def reset(engine, data, scene):
+def reset(engine, data, depsgraph):
     import _cycles
+    import bpy
+
+    if bpy.app.debug_value == 256:
+        _cycles.debug_flags_update(depsgraph.scene)
+    else:
+        _cycles.debug_flags_reset()
+
     data = data.as_pointer()
-    scene = scene.as_pointer()
-    _cycles.reset(engine.session, data, scene)
+    depsgraph = depsgraph.as_pointer()
+    _cycles.reset(engine.session, data, depsgraph)
 
 
-def update(engine, depsgraph, data, scene):
+def sync(engine, depsgraph, data):
     import _cycles
     _cycles.sync(engine.session, depsgraph.as_pointer())
 
diff --git a/intern/cycles/blender/blender_python.cpp 
b/intern/cycles/blender/blender_python.cpp
index 5c1181960c3..56c19c1827a 100644
--- a/intern/cycles/blender/blender_python.cpp
+++ b/intern/cycles/blender/blender_python.cpp
@@ -203,10 +203,10 @@ static PyObject *exit_func(PyObject * /*self*/, PyObject 
* /*args*/)
 
 static PyObject *create_func(PyObject * /*self*/, PyObject *args)
 {
-       PyObject *pyengine, *pyuserpref, *pydata, *pyscene, *pyregion, *pyv3d, 
*pyrv3d;
+       PyObject *pyengine, *pyuserpref, *pydata, *pyregion, *pyv3d, *pyrv3d;
        int preview_osl;
 
-       if(!PyArg_ParseTuple(args, "OOOOOOOi", &pyengine, &pyuserpref, &pydata, 
&pyscene,
+       if(!PyArg_ParseTuple(args, "OOOOOOi", &pyengine, &pyuserpref, &pydata,
                             &pyregion, &pyv3d, &pyrv3d, &preview_osl))
        {
                return NULL;
@@ -225,10 +225,6 @@ static PyObject *create_func(PyObject * /*self*/, PyObject 
*args)
        RNA_main_pointer_create((Main*)PyLong_AsVoidPtr(pydata), &dataptr);
        BL::BlendData data(dataptr);
 
-       PointerRNA sceneptr;
-       RNA_id_pointer_create((ID*)PyLong_AsVoidPtr(pyscene), &sceneptr);
-       BL::Scene scene(sceneptr);
-
        PointerRNA regionptr;
        RNA_pointer_create(NULL, &RNA_Region, 
pylong_as_voidptr_typesafe(pyregion), &regionptr);
        BL::Region region(regionptr);
@@ -249,27 +245,13 @@ static PyObject *create_func(PyObject * /*self*/, 
PyObject *args)
                int width = region.width();
                int height = region.height();
 
-               session = new BlenderSession(engine, userpref, data, scene, 
v3d, rv3d, width, height);
+               session = new BlenderSession(engine, userpref, data, v3d, rv3d, 
width, height);
        }
        else {
-               /* override some settings for preview */
-               if(engine.is_preview()) {
-                       PointerRNA cscene = RNA_pointer_get(&sceneptr, 
"cycles");
-
-                       RNA_boolean_set(&cscene, "shading_system", preview_osl);
-                       RNA_boolean_set(&cscene, "use_progressive_refine", 
true);
-               }
-
                /* offline session or preview render */
-               session = new BlenderSession(engine, userpref, data, scene);
+               session = new BlenderSession(engine, userpref, data, 
preview_osl);
        }
 
-       python_thread_state_save(&session->python_thread_state);
-
-       session->create();
-
-       python_thread_state_restore(&session->python_thread_state);
-
        return PyLong_FromVoidPtr(session);
 }
 
@@ -316,7 +298,7 @@ static PyObject *bake_func(PyObject * /*self*/, PyObject 
*args)
        BlenderSession *session = (BlenderSession*)PyLong_AsVoidPtr(pysession);
 
        PointerRNA depsgraphptr;
-       RNA_id_pointer_create((ID*)PyLong_AsVoidPtr(pydepsgraph), 
&depsgraphptr);
+       RNA_pointer_create(NULL, &RNA_Depsgraph, PyLong_AsVoidPtr(pydepsgraph), 
&depsgraphptr);
        BL::Depsgraph b_depsgraph(depsgraphptr);
 
        PointerRNA objectptr;
@@ -360,9 +342,9 @@ static PyObject *draw_func(PyObject * /*self*/, PyObject 
*args)
 
 static PyObject *reset_func(PyObject * /*self*/, PyObject *args)
 {
-       PyObject *pysession, *pydata, *pyscene;
+       PyObject *pysession, *pydata, *pydepsgraph;
 
-       if(!PyArg_ParseTuple(args, "OOO", &pysession, &pydata, &pyscene))
+       if(!PyArg_ParseTuple(args, "OOO", &pysession, &pydata, &pydepsgraph))
                return NULL;
 
        BlenderSession *session = (BlenderSession*)PyLong_AsVoidPtr(pysession);
@@ -371,13 +353,13 @@ static PyObject *reset_func(PyObject * /*self*/, PyObject 
*args)
        RNA_main_pointer_create((Main*)PyLong_AsVoidPtr(pydata), &dataptr);
        BL::BlendData b_data(dataptr);
 
-       PointerRNA sceneptr;
-       RNA_id_pointer_create((ID*)PyLong_AsVoidPtr(pyscene), &sceneptr);
-       BL::Scene b_scene(sceneptr);
+       PointerRNA depsgraphptr;
+       RNA_pointer_create(NULL, &RNA_Depsgraph, PyLong_AsVoidPtr(pydepsgraph), 
&depsgraphptr);
+       BL::Depsgraph b_depsgraph(depsgraphptr);
 
        python_thread_state_save(&session->python_thread_state);
 
-       session->reset_session(b_data, b_scene);
+       session->reset_session(b_data, b_depsgraph);
 
        python_thread_state_restore(&session->python_thread_state);
 
@@ -394,7 +376,7 @@ static PyObject *sync_func(PyObject * /*self*/, PyObject 
*args)
        BlenderSession *session = (BlenderSession*)PyLong_AsVoidPtr(pysession);
 
        PointerRNA depsgraphptr;
-       RNA_id_pointer_create((ID*)PyLong_AsVoidPtr(pydepsgraph), 
&depsgraphptr);
+       RNA_pointer_create(NULL, &RNA_Depsgraph, PyLong_AsVoidPtr(pydepsgraph), 
&depsgraphptr);
        BL::Depsgraph b_depsgraph(depsgraphptr);
 
        python_thread_state_save(&session->python_thread_state);
diff --git a/intern/cycles/blender/blender_session.cpp 
b/intern/cycles/blender/blender_session.cpp
index 9d55ece3e0c..bfaa843eb18 100644
--- a/intern/cycles/blender/blender_session.cpp
+++ b/intern/cycles/blender/blender_session.cpp
@@ -52,22 +52,22 @@ int BlenderSession::end_resumable_chunk = 0;
 BlenderSession::BlenderSession(BL::RenderEngine& b_engine,
                                BL::UserPreferences& b_userpref,
                                BL::BlendData& b_data,
-                               BL::Scene& b_scene)
-: b_engine(b_engine),
+                               bool preview_osl)
+: session(NULL),
+  b_engine(b_engine),
   b_userpref(b_userpref),
   b_data(b_data),
   b_render(b_engine.render()),
   b_depsgraph(PointerRNA_NULL),
-  b_scene(b_scene),
+  b_scene(PointerRNA_NULL),
   b_v3d(PointerRNA_NULL),
   b_rv3d(PointerRNA_NULL),
+  width(0),
+  height(0),
+  preview_osl(preview_osl),
   python_thread_state(NULL)
 {
        /* offline render */
-
-       width = render_resolution_x(b_render);
-       height = render_resolution_y(b_render);
-
        background = true;
        last_redraw_time = 0.0;
        start_resize_time = 0.0;
@@ -77,24 +77,24 @@ BlenderSession::BlenderSession(BL::RenderEngine& b_engine,
 BlenderSession::BlenderSession(BL::RenderEngine& b_engine,
                                BL::UserPreferences& b_userpref,
                                BL::BlendData& b_data,
-                               BL::Scene& b_scene,
                                BL::SpaceView3D& b_v3d,
                                BL::RegionView3D& b_rv3d,
                                int width, int height)
-: b_engine(b_engine),
+: session(NULL),
+  b_engine(b_engine),
   b_userpref(b_userpref),
   b_data(b_data),
-  b_render(b_scene.render()

@@ Diff output truncated at 10240 characters. @@

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

Reply via email to