Commit: 56021cad31c2cc17aa5abbbad1bf789563f530af
Author: Campbell Barton
Date:   Wed Oct 14 12:39:52 2015 +1100
Branches: master
https://developer.blender.org/rB56021cad31c2cc17aa5abbbad1bf789563f530af

Support for multi-sample sequencer GL render

OpenGL sequencer render now uses a single fbo for all rendering.

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

M       source/blender/blenkernel/BKE_sequencer.h
M       source/blender/blenkernel/intern/sequencer.c
M       source/blender/editors/include/ED_view3d.h
M       source/blender/editors/render/render_opengl.c
M       source/blender/editors/sculpt_paint/paint_image_proj.c
M       source/blender/editors/space_view3d/view3d_draw.c
M       source/blender/windowmanager/intern/wm_files.c

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

diff --git a/source/blender/blenkernel/BKE_sequencer.h 
b/source/blender/blenkernel/BKE_sequencer.h
index 360243d..06f4613 100644
--- a/source/blender/blenkernel/BKE_sequencer.h
+++ b/source/blender/blenkernel/BKE_sequencer.h
@@ -35,6 +35,7 @@ struct EvaluationContext;
 struct StripColorBalance;
 struct Editing;
 struct GSet;
+struct GPUOffScreen;
 struct ImBuf;
 struct Main;
 struct Mask;
@@ -101,6 +102,10 @@ typedef struct SeqRenderData {
        bool skip_cache;
        bool is_proxy_render;
        size_t view_id;
+
+       /* special case for OpenGL render */
+       struct GPUOffScreen *gpu_offscreen;
+       int gpu_samples;
 } SeqRenderData;
 
 void BKE_sequencer_new_render_data(
@@ -408,7 +413,11 @@ struct Sequence *BKE_sequencer_add_sound_strip(struct 
bContext *C, ListBase *seq
 struct Sequence *BKE_sequencer_add_movie_strip(struct bContext *C, ListBase 
*seqbasep, struct SeqLoadInfo *seq_load);
 
 /* view3d draw callback, run when not in background view */
-typedef struct ImBuf *(*SequencerDrawView)(struct Scene *, struct Object *, 
int, int, unsigned int, int, bool, bool, bool, int, const char *, char[256]);
+typedef struct ImBuf *(*SequencerDrawView)(
+        struct Scene *, struct Object *, int, int,
+        unsigned int, int, bool, bool, bool,
+        int, int, const char *,
+        struct GPUOffScreen *, char[256]);
 extern SequencerDrawView sequencer_view3d_cb;
 
 /* copy/paste */
diff --git a/source/blender/blenkernel/intern/sequencer.c 
b/source/blender/blenkernel/intern/sequencer.c
index fc8fa61..40cacb2 100644
--- a/source/blender/blenkernel/intern/sequencer.c
+++ b/source/blender/blenkernel/intern/sequencer.c
@@ -563,6 +563,8 @@ void BKE_sequencer_new_render_data(
        r_context->skip_cache = false;
        r_context->is_proxy_render = false;
        r_context->view_id = 0;
+       r_context->gpu_offscreen = NULL;
+       r_context->gpu_samples = (scene->r.mode & R_OSA) ? scene->r.osa : 0;
 }
 
 /* ************************* iterator ************************** */
@@ -3212,10 +3214,14 @@ static ImBuf *seq_render_scene_strip(const 
SeqRenderData *context, Sequence *seq
 
                /* opengl offscreen render */
                BKE_scene_update_for_newframe(context->eval_ctx, 
context->bmain, scene, scene->lay);
-               ibuf = sequencer_view3d_cb(scene, camera, width, height, 
IB_rect,
-                                          context->scene->r.seq_prev_type,
-                                          (context->scene->r.seq_flag & 
R_SEQ_SOLID_TEX) != 0,
-                                          use_gpencil, true, 
scene->r.alphamode, viewname, err_out);
+               ibuf = sequencer_view3d_cb(
+                       /* set for OpenGL render (NULL when scrubbing) */
+                       scene, camera, width, height, IB_rect,
+                       context->scene->r.seq_prev_type,
+                       (context->scene->r.seq_flag & R_SEQ_SOLID_TEX) != 0,
+                       use_gpencil, true, scene->r.alphamode,
+                       context->gpu_samples, viewname,
+                       context->gpu_offscreen, err_out);
                if (ibuf == NULL) {
                        fprintf(stderr, "seq_render_scene_strip failed to get 
opengl buffer: %s\n", err_out);
                }
diff --git a/source/blender/editors/include/ED_view3d.h 
b/source/blender/editors/include/ED_view3d.h
index 416e821..5cd73e4 100644
--- a/source/blender/editors/include/ED_view3d.h
+++ b/source/blender/editors/include/ED_view3d.h
@@ -323,15 +323,20 @@ bool ED_view3d_context_activate(struct bContext *C);
 void ED_view3d_draw_offscreen_init(struct Scene *scene, struct View3D *v3d);
 void ED_view3d_draw_offscreen(
         struct Scene *scene, struct View3D *v3d, struct ARegion *ar, int winx, 
int winy, float viewmat[4][4],
-        float winmat[4][4], bool do_bgpic, bool do_sky, bool is_persp,
-        struct GPUOffScreen *ofs,
+        float winmat[4][4], bool do_bgpic, bool do_sky, bool is_persp, const 
char *viewname,
         struct GPUFX *fx, struct GPUFXSettings *fx_settings,
-        const char *viewname);
-
-struct ImBuf *ED_view3d_draw_offscreen_imbuf(struct Scene *scene, struct 
View3D *v3d, struct ARegion *ar, int sizex, int sizey, unsigned int flag,
-                                             bool draw_background, int 
alpha_mode, const char *viewname, char err_out[256]);
-struct ImBuf *ED_view3d_draw_offscreen_imbuf_simple(struct Scene *scene, 
struct Object *camera, int width, int height, unsigned int flag, int drawtype,
-                                             bool use_solid_tex, bool 
use_gpencil, bool draw_background, int alpha_mode, const char *viewname, char 
err_out[256]);
+        struct GPUOffScreen *ofs);
+
+struct ImBuf *ED_view3d_draw_offscreen_imbuf(
+        struct Scene *scene, struct View3D *v3d, struct ARegion *ar, int 
sizex, int sizey,
+        unsigned int flag, bool draw_background,
+        int alpha_mode, int samples, const char *viewname,
+        struct GPUOffScreen *ofs, char err_out[256]);
+struct ImBuf *ED_view3d_draw_offscreen_imbuf_simple(
+        struct Scene *scene, struct Object *camera, int width, int height,
+        unsigned int flag, int drawtype, bool use_solid_tex, bool use_gpencil, 
bool draw_background,
+        int alpha_mode, int samples, const char *viewname,
+        struct GPUOffScreen *ofs, char err_out[256]);
 
 struct Base *ED_view3d_give_base_under_cursor(struct bContext *C, const int 
mval[2]);
 void ED_view3d_quadview_update(struct ScrArea *sa, struct ARegion *ar, bool 
do_clip);
diff --git a/source/blender/editors/render/render_opengl.c 
b/source/blender/editors/render/render_opengl.c
index c8ced55..46ed860 100644
--- a/source/blender/editors/render/render_opengl.c
+++ b/source/blender/editors/render/render_opengl.c
@@ -96,6 +96,7 @@ typedef struct OGLRender {
        ImageUser iuser;
 
        GPUOffScreen *ofs;
+       int ofs_samples;
        GPUFX *fx;
        int sizex, sizey;
        int write_still;
@@ -279,6 +280,9 @@ static void screen_opengl_render_doit(OGLRender *oglrender, 
RenderResult *rr)
                        &context);
 
                context.view_id = BKE_scene_multiview_view_id_get(&scene->r, 
viewname);
+               context.gpu_offscreen = oglrender->ofs;
+               context.gpu_samples = oglrender->ofs_samples;
+
                ibuf = BKE_sequencer_give_ibuf(&context, CFRA, chanshown);
 
                if (ibuf) {
@@ -377,8 +381,9 @@ static void screen_opengl_render_doit(OGLRender *oglrender, 
RenderResult *rr)
 
                ED_view3d_draw_offscreen(
                        scene, v3d, ar, sizex, sizey, NULL, winmat,
-                       draw_bgpic, draw_sky, is_persp,
-                       oglrender->ofs, oglrender->fx, &fx_settings, viewname);
+                       draw_bgpic, draw_sky, is_persp, viewname,
+                       oglrender->fx, &fx_settings,
+                       oglrender->ofs);
                GPU_offscreen_read_pixels(oglrender->ofs, GL_UNSIGNED_BYTE, 
rect);
 
                GPU_offscreen_unbind(oglrender->ofs, true); /* unbind */
@@ -386,9 +391,11 @@ static void screen_opengl_render_doit(OGLRender 
*oglrender, RenderResult *rr)
        else {
                /* shouldnt suddenly give errors mid-render but possible */
                char err_out[256] = "unknown";
-               ImBuf *ibuf_view = ED_view3d_draw_offscreen_imbuf_simple(scene, 
scene->camera, oglrender->sizex, oglrender->sizey,
-                                                                        
IB_rect, OB_SOLID, false, true, true,
-                                                                        
(draw_sky) ? R_ADDSKY : R_ALPHAPREMUL, viewname, err_out);
+               ImBuf *ibuf_view = ED_view3d_draw_offscreen_imbuf_simple(
+                       scene, scene->camera, oglrender->sizex, 
oglrender->sizey,
+                       IB_rect, OB_SOLID, false, true, true,
+                       (draw_sky) ? R_ADDSKY : R_ALPHAPREMUL, 
oglrender->ofs_samples, viewname,
+                       oglrender->ofs, err_out);
                camera = scene->camera;
 
                if (ibuf_view) {
@@ -498,12 +505,12 @@ static bool screen_opengl_render_init(bContext *C, 
wmOperator *op)
        GPUOffScreen *ofs;
        OGLRender *oglrender;
        int sizex, sizey;
+       const int samples = (scene->r.mode & R_OSA) ? scene->r.osa : 0;
        bool is_view_context = RNA_boolean_get(op->ptr, "view_context");
        const bool is_animation = RNA_boolean_get(op->ptr, "animation");
        const bool is_sequencer = RNA_boolean_get(op->ptr, "sequencer");
        const bool is_write_still = RNA_boolean_get(op->ptr, "write_still");
        char err_out[256] = "unknown";
-       int samples = (scene->r.mode & R_OSA) ? scene->r.osa : 0;
 
        if (G.background) {
                BKE_report(op->reports, RPT_ERROR, "Cannot use OpenGL render in 
background mode (no opengl context)");
@@ -555,6 +562,7 @@ static bool screen_opengl_render_init(bContext *C, 
wmOperator *op)
        op->customdata = oglrender;
 
        oglrender->ofs = ofs;
+       oglrender->ofs_samples = samples;
        oglrender->sizex = sizex;
        oglrender->sizey = sizey;
        oglrender->bmain = CTX_data_main(C);
diff --git a/source/blender/editors/sculpt_paint/paint_image_proj.c 
b/source/blender/editors/sculpt_paint/paint_image_proj.c
index 73a3cce..565f8a5 100644
--- a/source/blender/editors/sculpt_paint/paint_image_proj.c
+++ b/source/blender/editors/sculpt_paint/paint_image_proj.c
@@ -5399,7 +5399,10 @@ static int texture_paint_image_from_view_exec(bContext 
*C, wmOperator *op)
        if (w > maxsize) w = maxsize;
        if (h > maxsize) h = maxsize;
 
-       ibuf = ED_view3d_draw_offscreen_imbuf(scene, CTX_wm_view3d(C), 
CTX_wm_region(C), w, h, IB_rect, false, R_ALPHAPREMUL, NULL, err_out);
+       ibuf = ED_view3d_draw_offscreen_imbuf(
+               scene, CTX_wm_view3d(C), CTX_wm_region(C),
+               w, h, IB_rect, false, R_ALPHAPREMUL, 0, NULL,
+               NULL, err_out);
        if (!ibuf) {
                /* Mostly happens when OpenGL offscreen buffer was failed to 
create, */
                /* but could be other reasons. Should be handled in the future. 
nazgul */
diff --git a/source/blender/editors/space_view3d/view3d_draw.c 
b/source/blender/editors/space_view3d/view3d_draw.c
index ac12528..6840e8a 100644
--- a/source/blender/editors/space_view3d/view3d_draw.c
+++ b/source/blender/editors/space_view3d/view3d_draw.c
@@ -3144,10 +3144,9 @@ static void view3d_main_area_clear(Scene *scene, View3D 
*v3d, ARegion *ar)
 void ED_view3d_draw_offscreen(
         Scene *scene, View3D *v3d, ARegion *ar, int winx, int winy,
         float viewmat[4][4], float winmat[4][4],
-        bool do_bgpic, bool do_sky, bool is_persp,
-        GPUOffScreen *ofs,
+        bool do_bgpic, bool do_sky, bool is_persp, const char *viewname,
         GPUFX *fx, GPUFXSettings *fx_settings,
-        const char *viewname)
+        GPUOffScreen *ofs)
 {
        struct bThemeState theme_state;
        int bwinx, bwiny;
@@ -3247,26 +3246,37 @@ void E

@@ Diff output truncated at 10240 characters. @@

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

Reply via email to