Bench: inplace draw vs copy draw.

src/pixie/images.nim

@@ -238,6 +238,21 @@ proc hasEffect*(blendMode: BlendMode, rgba: ColorRGBA): bool =
   else:
     rgba.a > 0
 
+proc drawBlendIntegerPos*(
+  destImage, srcImage: Image, pos = vec2(0, 0), blendMode = Normal,
+) =
+  ## Fast draw of dest + fill using offset with color blending.
+  for y in 0 ..< srcImage.height:
+    for x in 0 ..< srcImage.width:
+      let
+        srcRgba = srcImage.getRgbaUnsafe(x, y)
+      if blendMode.hasEffect(srcRgba):
+        let
+          destRgba = destImage.getRgbaUnsafe(x + pos.x.int, y + pos.y.int)
+          rgba = blendMode.mix(destRgba, srcRgba)
+        # TODO: Make unsafe
+        destImage[x + pos.x.int, y + pos.y.int] = rgba
+
 proc draw*(destImage: Image, srcImage: Image, mat: Mat4, blendMode = Normal) =
   ## Draws one image onto another using matrix with color blending.
   var srcImage = srcImage
@@ -280,28 +295,34 @@ proc draw*(destImage: Image, srcImage: Image, mat: Mat4, blendMode = Normal) =
   for y in yStart ..< yEnd:
     for x in xStart ..< xEnd:
       let srcV = start + stepX * float32(x) + stepY * float32(y)
-      if srcImage.inside(int srcV.x, int srcV.y):
+      if srcImage.inside(int srcV.x.floor, int srcV.y.floor):
         let
           srcRgba = srcImage.getRgbaSmooth(srcV.x - 0.5, srcV.y - 0.5)
-        let
+        if blendMode.hasEffect(srcRgba):
-          destRgba = destImage.getRgbaUnsafe(x, y)
+          let
-          color = blendMode.mix(destRgba.color, srcRgba.color)
+            destRgba = destImage.getRgbaUnsafe(x, y)
-        destImage.setRgbaUnsafe(x, y, color.rgba)
+            color = blendMode.mix(destRgba, srcRgba)
+          destImage.setRgbaUnsafe(x, y, color)
 
-proc draw*(
+proc gpuDraw*(destImage: Image, srcImage: Image, mat: Mat3, blendMode = Normal): Image =
-  destImage, srcImage: Image, pos = vec2(0, 0), blendMode = Normal,
+  ## Draws one image onto another using matrix with color blending.
-) =
+  result = newImage(destImage.width, destImage.height)
-  ## Fast draw of dest + fill using offset with color blending.
+  for y in 0 .. result.width:
-  for y in 0 ..< srcImage.height:
+    for x in 0 .. result.height:
-    for x in 0 ..< srcImage.width:
+      let srcPos = mat * vec2(x.float32, y.float32)
-      let
+      var destRgba = destImage.getRgbaUnsafe(x, y)
-        srcRgba = srcImage.getRgbaUnsafe(x, y)
+      var srcRgba = rgba(0, 0, 0, 0)
+      if srcImage.inside(srcPos.x.floor.int, srcPos.y.floor.int):
+        srcRgba = srcImage.getRgbaSmooth(srcPos.x - 0.5, srcPos.y - 0.5)
       if blendMode.hasEffect(srcRgba):
-        let
+        destRgba = blendMode.mix(destRgba, srcRgba)
-          destRgba = destImage.getRgbaUnsafe(x + pos.x.int, y + pos.y.int)
+      result.setRgbaUnsafe(x, y, destRgba)
-          rgba = blendMode.mix(destRgba, srcRgba)
+
-        # TODO: Make unsafe
+proc draw*(destImage: Image, srcImage: Image, pos = vec2(0, 0), blendMode = Normal) =
-        destImage[x + pos.x.int, y + pos.y.int] = rgba
+  destImage.draw(srcImage, translate(vec3(pos.x, pos.y, 0)), blendMode)
+
+proc gpuDraw*(destImage: Image, srcImage: Image, pos = vec2(0, 0), blendMode = Normal): Image =
+  destImage.gpuDraw(srcImage, translate(-pos), blendMode)
 
 ## Thoughts
 ## single draw function that takes a matrix

tests/benchmark_images.nim

@@ -0,0 +1,40 @@
+import pixie, chroma, vmath, fidget/opengl/perf, pixie/fileformats/bmp
+
+block:
+  var a = newImage(100, 100)
+  a.fill(rgba(255, 0, 0, 255))
+  var b = newImage(100, 100)
+  b.fill(rgba(0, 255, 0, 255))
+  a.draw(b, pos=vec2(25, 25))
+  writeFile("tests/images/inplaceDraw.bmp", a.encodeBmp())
+
+block:
+  var a = newImage(100, 100)
+  a.fill(rgba(255, 0, 0, 255))
+  var b = newImage(100, 100)
+  b.fill(rgba(0, 255, 0, 255))
+  var c = a.gpuDraw(b, pos=vec2(25, 25))
+  writeFile("tests/images/copyDraw.bmp", c.encodeBmp())
+
+
+timeIt "inplaceDraw":
+  var tmp = 0
+  for i in 0 ..< 1000:
+    var a = newImage(100, 100)
+    a.fill(rgba(255, 0, 0, 255))
+    var b = newImage(100, 100)
+    b.fill(rgba(0, 255, 0, 255))
+    a.draw(b, pos=vec2(25, 25))
+    tmp += a.width * a.height
+  echo tmp
+
+timeIt "copyDraw":
+  var tmp = 0
+  for i in 0 ..< 1000:
+    var a = newImage(100, 100)
+    a.fill(rgba(255, 0, 0, 255))
+    var b = newImage(100, 100)
+    b.fill(rgba(0, 255, 0, 255))
+    var c = a.gpuDraw(b, pos=vec2(25, 25))
+    tmp += c.width * c.height
+  echo tmp