fix low hanging fruits for render performance

CairoMakie/src/infrastructure.jl

@@ -12,9 +12,7 @@ function cairo_draw(screen::Screen, scene::Scene)
     draw_background(screen, scene)
 
     allplots = Makie.collect_atomic_plots(scene; is_atomic_plot = is_cairomakie_atomic_plot)
-    zvals = Makie.zvalue2d.(allplots)
-    permute!(allplots, sortperm(zvals))
-
+    sort!(allplots; by=Makie.zvalue2d)
     # If the backend is not a vector surface (i.e., PNG/ARGB),
     # then there is no point in rasterizing twice.
     should_rasterize = is_vector_backend(screen.surface)
@@ -120,7 +118,7 @@ function draw_background(screen::Screen, scene::Scene, root_h)
         bg = scene.backgroundcolor[]
         Cairo.set_source_rgba(cr, red(bg), green(bg), blue(bg), alpha(bg));
         r = viewport(scene)[]
-        # Makie has (0,0) at bottom left, Cairo at top left. Makie extends up, 
+        # Makie has (0,0) at bottom left, Cairo at top left. Makie extends up,
         # Cairo down. Negative height breaks other backgrounds
         x, y = origin(r); w, h = widths(r)
         Cairo.rectangle(cr, x, root_h - y - h, w, h) # background

GLMakie/src/GLAbstraction/GLRenderObject.jl

@@ -68,14 +68,14 @@ function (sp::StandardPostrender)()
     render(sp.vao, sp.primitive)
 end
 
-struct StandardPostrenderInstanced{T}
-    main::T
+struct StandardPostrenderInstanced
+    n_instances::Observable{Int}
     vao::GLVertexArray
     primitive::GLenum
 end
 
 function (sp::StandardPostrenderInstanced)()
-    renderinstanced(sp.vao, to_value(sp.main), sp.primitive)
+    return renderinstanced(sp.vao, sp.n_instances[], sp.primitive)
 end
 
 struct EmptyPrerender end

GLMakie/src/postprocessing.jl

@@ -290,7 +290,7 @@ function to_screen_postprocessor(framebuffer, shader_cache, screen_fb_id = nothi
         default_id = isnothing(screen_fb_id) ? 0 : screen_fb_id[]
         # GLFW uses 0, Gtk uses a value that we have to probe at the beginning of rendering
         glBindFramebuffer(GL_FRAMEBUFFER, default_id)
-        glViewport(0, 0, framebuffer_size(screen.glscreen)...)
+        glViewport(0, 0, framebuffer_size(screen)...)
         glClear(GL_COLOR_BUFFER_BIT)
         GLAbstraction.render(pass) # copy postprocess
     end

GLMakie/src/rendering.jl

@@ -29,14 +29,12 @@ Renders a single frame of a `window`
 function render_frame(screen::Screen; resize_buffers=true)
     nw = to_native(screen)
     ShaderAbstractions.switch_context!(nw)
+
     function sortby(x)
-        robj = x[3]
-        plot = screen.cache2plot[robj.id]
-        # TODO, use actual boundingbox
-        return Makie.zvalue2d(plot)
+        return x[3][:model][][3, 4]
     end
-    zvals = sortby.(screen.renderlist)
-    permute!(screen.renderlist, sortperm(zvals))
+
+    sort!(screen.renderlist; by=sortby)
 
     # NOTE
     # The transparent color buffer is reused by SSAO and FXAA. Changing the

GLMakie/src/screen.jl

@@ -174,7 +174,7 @@ mutable struct Screen{GLWindow} <: MakieScreen
     renderlist::Vector{Tuple{ZIndex, ScreenID, RenderObject}}
     postprocessors::Vector{PostProcessor}
     cache::Dict{UInt64, RenderObject}
-    cache2plot::Dict{UInt32, AbstractPlot}
+    cache2plot::Dict{UInt32, Plot}
     framecache::Matrix{RGB{N0f8}}
     render_tick::Observable{Makie.TickState} # listeners must not Consume(true)
     window_open::Observable{Bool}
@@ -218,6 +218,8 @@ mutable struct Screen{GLWindow} <: MakieScreen
     end
 end
 
+framebuffer_size(screen::Screen) = screen.framebuffer.resolution[]
+
 Makie.isvisible(screen::Screen) = screen.config.visible
 
 # for e.g. closeall, track all created screens