summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFrederico Linhares <fred@linhares.blue>2023-06-14 17:16:37 -0300
committerFrederico Linhares <fred@linhares.blue>2023-06-17 16:41:47 -0300
commitc4c96f51d36f47a2c7eaefa9f2537cf7ff2c827c (patch)
tree5aecec86dca4bd69160093d2a969c4a1371113d0
parent482d53c89c90ee2a74d14c1a3e6c2fdab4ce0004 (diff)
feat Add z index when rendering sprites
* src/sprite.cpp: Add a new parameter to sprites to define the z index of the image being rendered. * src/vk/graphics_pipeline_2d_solid.cpp: Sort may not be the most efficient algorithm, but it is easier to implement. If this code becomes slow, it can be optimized with little or no changes to the Ruby interface.
-rw-r--r--src/sprite.cpp15
-rw-r--r--src/view_2d.hpp2
-rw-r--r--src/vk/graphics_pipeline_2d_solid.cpp38
-rw-r--r--src/vk/sprite_to_draw.cpp40
-rw-r--r--src/vk/sprite_to_draw.hpp44
-rw-r--r--src/vk/view_2d.hpp6
6 files changed, 116 insertions, 29 deletions
diff --git a/src/sprite.cpp b/src/sprite.cpp
index 8f25f03..dba9e22 100644
--- a/src/sprite.cpp
+++ b/src/sprite.cpp
@@ -1,5 +1,5 @@
/*
- * Copyright 2022 Frederico de Oliveira Linhares
+ * Copyright 2022-2023 Frederico de Oliveira Linhares
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -63,17 +63,17 @@ cg_cSprite_draw(mrb_state *mrb, mrb_value self)
{
mrb_value view_value;
VK::View2D *view_2d;
- mrb_float x, y, w, h;
+ mrb_float x, y, w, h, z_index{0.0};
auto ptr = (std::shared_ptr<VK::Sprite>*)DATA_PTR(self);
- mrb_get_args(mrb, "offff", &view_value, &x, &y, &w, &h);
+ mrb_get_args(mrb, "offff|f", &view_value, &x, &y, &w, &h, &z_index);
view_2d = cg_cView_to_view_2d(mrb, view_value);
glm::vec4 rect(x, y, x + w, y + h);
- auto &positions = view_2d->sprites_to_draw[
- cg_core.vk_swapchain->current_frame][*ptr];
- positions.push_back(rect);
+ auto &sprites_to_draw = view_2d->sprites_to_draw[
+ cg_core.vk_swapchain->current_frame];
+ sprites_to_draw.emplace_back(*ptr, rect, z_index);
return self;
}
@@ -88,5 +88,6 @@ cg_sprite_init(mrb_state *mrb)
MRB_SET_INSTANCE_TT(cg_cSprite, MRB_TT_DATA);
mrb_define_method(
mrb, cg_cSprite, "initialize", cg_cSprite_initialize, MRB_ARGS_REQ(5));
- mrb_define_method(mrb, cg_cSprite, "draw", cg_cSprite_draw, MRB_ARGS_REQ(5));
+ mrb_define_method(mrb, cg_cSprite, "draw", cg_cSprite_draw, MRB_ARGS_REQ(5)|
+ MRB_ARGS_OPT(1));
}
diff --git a/src/view_2d.hpp b/src/view_2d.hpp
index c2b1701..f2ff5b6 100644
--- a/src/view_2d.hpp
+++ b/src/view_2d.hpp
@@ -1,5 +1,5 @@
/*
- * Copyright 2022 Frederico de Oliveira Linhares
+ * Copyright 2022-2023 Frederico de Oliveira Linhares
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
diff --git a/src/vk/graphics_pipeline_2d_solid.cpp b/src/vk/graphics_pipeline_2d_solid.cpp
index 5d210be..380bb65 100644
--- a/src/vk/graphics_pipeline_2d_solid.cpp
+++ b/src/vk/graphics_pipeline_2d_solid.cpp
@@ -1,5 +1,5 @@
/*
- * Copyright 2022 Frederico de Oliveira Linhares
+ * Copyright 2022-2023 Frederico de Oliveira Linhares
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -299,33 +299,37 @@ GraphicsPipeline2DSolid::draw(
vkCmdSetScissor(draw_command_buffer, 0, 1, &vk_scissor);
}
+ vkCmdBindPipeline(
+ draw_command_buffer, VK_PIPELINE_BIND_POINT_GRAPHICS,
+ this->graphic_pipeline);
+
+ // FIXME: I know sorting is expensive, but I need to figure out better what
+ // to do here.
+ std::sort(view->sprites_to_draw[current_frame].rbegin(),
+ view->sprites_to_draw[current_frame].rend());
+
// Draw sprites
- for(auto& [sprite, positions]: view->sprites_to_draw[current_frame])
+ for(auto& sprite_to_draw: view->sprites_to_draw[current_frame])
{
std::array<VkDescriptorSet, 2> vk_descriptor_sets{
view->descriptor_sets_2d[image_index],
- sprite->descriptor_sets[image_index]};
+ sprite_to_draw.sprite->descriptor_sets[image_index]};
VkDeviceSize offsets[]{0};
vkCmdBindDescriptorSets(
draw_command_buffer, VK_PIPELINE_BIND_POINT_GRAPHICS,
cg_core.vk_graphics_pipeline_2d_solid_layout->pipeline, 0,
vk_descriptor_sets.size(), vk_descriptor_sets.data(), 0, nullptr);
- vkCmdBindPipeline(
- draw_command_buffer, VK_PIPELINE_BIND_POINT_GRAPHICS,
- this->graphic_pipeline);
vkCmdBindVertexBuffers(
- draw_command_buffer, 0, 1, &sprite->vertex_buffer->buffer, offsets);
-
- for(auto i{0}; i < positions.size(); i++)
- {
- ODOVector4D position{positions[i]};
- vkCmdPushConstants(
- draw_command_buffer,
- cg_core.vk_graphics_pipeline_2d_solid_layout->pipeline,
- VK_SHADER_STAGE_VERTEX_BIT, 0, sizeof(ODOVector4D), &position);
- vkCmdDraw(draw_command_buffer, Sprite::vertex_count, 1, 0, 0);
- }
+ draw_command_buffer, 0, 1, &sprite_to_draw.sprite->vertex_buffer->buffer,
+ offsets);
+
+ ODOVector4D position{sprite_to_draw.position};
+ vkCmdPushConstants(
+ draw_command_buffer,
+ cg_core.vk_graphics_pipeline_2d_solid_layout->pipeline,
+ VK_SHADER_STAGE_VERTEX_BIT, 0, sizeof(ODOVector4D), &position);
+ vkCmdDraw(draw_command_buffer, Sprite::vertex_count, 1, 0, 0);
}
// Prepare for the next frame.
diff --git a/src/vk/sprite_to_draw.cpp b/src/vk/sprite_to_draw.cpp
new file mode 100644
index 0000000..76ee847
--- /dev/null
+++ b/src/vk/sprite_to_draw.cpp
@@ -0,0 +1,40 @@
+/*
+ * Copyright 2022-2023 Frederico de Oliveira Linhares
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include "sprite_to_draw.hpp"
+
+namespace VK
+{
+ SpriteToDraw::SpriteToDraw(
+ std::shared_ptr<Sprite> sprite, const glm::vec4 &position, float z_index):
+ sprite{sprite},
+ position{position},
+ z_index{z_index}
+ {
+ }
+
+ bool
+ operator<(const SpriteToDraw& a, const SpriteToDraw& b)
+ {
+ return a.z_index < b.z_index;
+ }
+
+ bool
+ operator>(const SpriteToDraw& a, const SpriteToDraw& b)
+ {
+ return a.z_index > b.z_index;
+ }
+}
diff --git a/src/vk/sprite_to_draw.hpp b/src/vk/sprite_to_draw.hpp
new file mode 100644
index 0000000..3bd6af3
--- /dev/null
+++ b/src/vk/sprite_to_draw.hpp
@@ -0,0 +1,44 @@
+/*
+ * Copyright 2022-2023 Frederico de Oliveira Linhares
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#ifndef CANDY_GEAR_VK_SPRITES_TO_DRAW_2D_H
+#define CANDY_GEAR_VK_SPRITES_TO_DRAW_2D_H 1
+
+#include <memory>
+
+#include "core.hpp"
+#include "sprite.hpp"
+
+namespace VK
+{
+ struct SpriteToDraw
+ {
+ std::shared_ptr<Sprite> sprite;
+ glm::vec4 position;
+ float z_index;
+
+ SpriteToDraw(std::shared_ptr<Sprite> sprite, const glm::vec4 &position,
+ float z_index);
+ };
+
+ bool
+ operator<(const SpriteToDraw &a, const SpriteToDraw &b);
+
+ bool
+ operator<(const SpriteToDraw &a, const SpriteToDraw &b);
+}
+
+#endif /* CANDY_GEAR_VK_SPRITES_TO_DRAW_2D_H */
diff --git a/src/vk/view_2d.hpp b/src/vk/view_2d.hpp
index caa3d0f..ead21e2 100644
--- a/src/vk/view_2d.hpp
+++ b/src/vk/view_2d.hpp
@@ -22,7 +22,7 @@
#include <vector>
#include "core.hpp"
-#include "sprite.hpp"
+#include "sprite_to_draw.hpp"
#include "rectangle.hpp"
namespace VK
@@ -40,9 +40,7 @@ struct View2D
std::vector<VkDescriptorSet> descriptor_sets_2d;
std::vector<std::vector<Rectangle>> rectangles_to_draw;
- std::vector<
- std::unordered_map<std::shared_ptr<Sprite>, std::vector<glm::vec4>>>
- sprites_to_draw;
+ std::vector<std::vector<SpriteToDraw>> sprites_to_draw;
View2D(glm::vec4 region, float projection_width, float projection_height);
virtual ~View2D();