move sky sprites to environment

src/graphics/render/Skybox.cpp

@@ -23,7 +23,6 @@
 using namespace advanced_pipeline;
 
 const int STARS_COUNT = 3000;
-
 const int STARS_SEED = 632;
 
 Skybox::Skybox(uint size, const Assets& assets)
@@ -41,31 +40,6 @@ Skybox::Skybox(uint size, const Assets& assets)
         {{1.0f, -1.0f}}};
 
     mesh = std::make_unique<Mesh<SkyboxVertex>>(vertices, 6);
-
-    sprites.push_back(SkySprite {
-        "misc/moon",
-        glm::pi<float>() * 0.5f,
-        4.0f,
-        false,
-        glm::pi<float>() * 0.25f,
-    });
-
-    sprites.push_back(SkySprite {
-        "misc/moon_flare",
-        glm::pi<float>() * 0.5f,
-        0.5f,
-        false,
-        glm::pi<float>() * 0.25f,
-    });
-
-    sprites.push_back(SkySprite {
-        "misc/sun",
-        glm::pi<float>() * 1.5f,
-        4.0f,
-        true,
-        glm::pi<float>() * 0.25f,
-    });
-
     setMode(mode);
 }
 
@@ -117,7 +91,8 @@ void Skybox::drawStars(float angle, float opacity) {
 void Skybox::drawSkySprites(
     float daytime,
     float angle,
-    float opacity
+    float opacity,
+    const std::vector<SkySprite>& sprites
 ) {
     float depthScale = 2e3;
     for (auto& sprite : sprites) {
@@ -160,7 +135,7 @@ void Skybox::draw(
     DrawContext ctx = pctx.sub();
     ctx.setBlendMode(BlendMode::addition);
 
-    if (!environment.sky.sprites && !environment.sky.stars) {
+    if (environment.sky.sprites.empty() && !environment.sky.stars) {
         return;
     }
 
@@ -170,8 +145,8 @@ void Skybox::draw(
     shader->uniformMatrix("u_apply", glm::mat4(1.0f));
     batch3d->begin();
 
-    if (environment.sky.sprites) {
+    if (!environment.sky.sprites.empty()) {
-        drawSkySprites(daytime, angle, opacity);
+        drawSkySprites(daytime, angle, opacity, environment.sky.sprites);
     }
     if (environment.sky.stars) {
         drawStars(angle, opacity);

src/graphics/render/Skybox.hpp

@@ -10,7 +10,6 @@
 #include "typedefs.hpp"
 #include "world/Environment.hpp"
 
-
 template <typename VertexStructure>
 class Mesh;
 class Shader;
@@ -28,14 +27,6 @@ struct SkyboxVertex {
         {VertexAttribute::Type::FLOAT, false, 2}, {{}, 0}};
 };
 
-struct SkySprite {
-    std::string texture;
-    float phase;
-    float distance;
-    bool emissive;
-    float altitude;
-};
-
 class Skybox {
     SkyMode mode = SkyMode::SOLID;
     uint size;
@@ -47,7 +38,6 @@ class Skybox {
 
     std::unique_ptr<Mesh<SkyboxVertex>> mesh;
     std::unique_ptr<Batch3D> batch3d;
-    std::vector<SkySprite> sprites;
     int frameid = 0;
 
     float prevMie = -1.0f;
@@ -58,7 +48,12 @@ class Skybox {
 
     void drawStars(float angle, float opacity);
     void drawBackground(const Camera& camera, int width, int height);
-    void drawSkySprites(float daytime, float angle, float opacity);
+    void drawSkySprites(
+        float daytime,
+        float angle,
+        float opacity,
+        const std::vector<SkySprite>& sprites
+    );
     void refreshFace(uint face, Cubemap& cubemap);
 public:
     Skybox(uint size, const Assets& assets);

src/world/Environment.cpp

@@ -5,12 +5,22 @@
 #include "data/dv_util.hpp"
 
 dv::value Environment::serialize() const {
+    auto skySpritesList = dv::list();
+    for (const auto& sprite : sky.sprites) {
+        skySpritesList.add(dv::object({
+            {"texture", sprite.texture},
+            {"phase", sprite.phase},
+            {"distance", sprite.distance},
+            {"emissive", sprite.emissive},
+            {"altitude", sprite.altitude},
+        }));
+    }
     return dv::object({
         {"sky", dv::object({
             {"mode", SkyModeMeta.getNameString(sky.mode)},
             {"stars", sky.stars},
             {"clouds", sky.clouds},
-            {"sprites", sky.sprites},
+            {"sprites", std::move(skySpritesList)},
         })},
     });
 }
@@ -20,6 +30,18 @@ void Environment::deserialize(const dv::value& src) {
         skyItem->at("mode").get(sky.mode, SkyModeMeta);
         skyItem->at("stars").get(sky.stars);
         skyItem->at("clouds").get(sky.clouds);
-        skyItem->at("sprites").get(sky.sprites);
+
+        if (auto skySpritesList = skyItem->at("sprites")) {
+            for (int i = 0; i < skySpritesList->size(); i++) {
+                SkySprite sprite;
+                const auto& item = (*skySpritesList.ptr)[i];
+                item.at("texture").get(sprite.texture);
+                item.at("phase").get(sprite.phase);
+                item.at("distance").get(sprite.distance);
+                item.at("emissive").get(sprite.emissive);
+                item.at("altitude").get(sprite.altitude);
+                sky.sprites.push_back(std::move(sprite));
+            }
+        }
     }
 }

src/world/Environment.hpp

@@ -3,6 +3,9 @@
 #include "interfaces/Serializable.hpp"
 #include "util/EnumMetadata.hpp"
 
+#include <vector>
+#include <glm/gtc/constants.hpp>
+
 enum class SkyMode {
     NONE, SOLID, BOX
 };
@@ -13,13 +16,44 @@ VC_ENUM_METADATA(SkyMode)
     {"box", SkyMode::BOX},
 VC_ENUM_END
 
+struct SkySprite {
+    std::string texture;
+    float phase;
+    float distance;
+    bool emissive;
+    float altitude;
+};
+
 class Environment : public Serializable {
 public:
     struct {
         SkyMode mode = SkyMode::BOX;
         bool stars = true;
         bool clouds = true;
-        bool sprites = true;
+        // TODO: move defaults to config file
+        std::vector<SkySprite> sprites {
+            SkySprite {
+                "misc/moon",
+                glm::pi<float>() * 0.5f,
+                4.0f,
+                false,
+                glm::pi<float>() * 0.25f,
+            },
+            SkySprite {
+                "misc/moon_flare",
+                glm::pi<float>() * 0.5f,
+                0.5f,
+                false,
+                glm::pi<float>() * 0.25f,
+            },
+            SkySprite {
+                "misc/sun",
+                glm::pi<float>() * 1.5f,
+                4.0f,
+                true,
+                glm::pi<float>() * 0.25f,
+            },
+        };
     } sky;
 
     Environment() = default;