Merge pull request 'Add support for MSDF font rendering' (#109) from msdf_text into master

Reviewed-on: https://git.madvoxel.net/OpenVulkano/OpenVulkano/pulls/109
Reviewed-by: Georg Hagen <georg.hagen@madvoxel.com>

.clang-format

@@ -4,10 +4,10 @@ AccessModifierOffset: "-4"
 AlignAfterOpenBracket: Align
 AlignEscapedNewlines: Left
 AlignOperands: true
-AllowShortBlocksOnASingleLine: Always
+AllowShortBlocksOnASingleLine: Empty
-AllowShortCaseLabelsOnASingleLine: true
+AllowShortCaseLabelsOnASingleLine: false
-AllowShortIfStatementsOnASingleLine: AllIfsAndElse
+AllowShortIfStatementsOnASingleLine: Never
-AllowShortLoopsOnASingleLine: true
+AllowShortLoopsOnASingleLine: false
 BreakBeforeBinaryOperators: NonAssignment
 BreakBeforeBraces: Allman
 BreakBeforeTernaryOperators: false
@@ -23,6 +23,7 @@ IndentCaseLabels: true
 IndentPPDirectives: BeforeHash
 IndentWidth: "4"
 IndentWrappedFunctionNames: true
+InsertBraces: true
 KeepEmptyLinesAtEOF: true
 KeepEmptyLinesAtTheStartOfBlocks: false
 MaxEmptyLinesToKeep: "2"

examples/ExampleApps/TextExampleApp.cpp

@@ -24,7 +24,7 @@
 #include "Controller/FreeCamCameraController.hpp"
 #include "Image/ImageLoaderPng.hpp"
 #include "Scene/FontAtlasGenerator.hpp"
-#include "Scene/MsdfFontAtlasGenerator.hpp"
+#include "Scene/IFontAtlasGenerator.hpp"
 #include <filesystem>
 
 #ifdef _WIN32
@@ -57,7 +57,7 @@ namespace OpenVulkano
 			texts.push_back(std::make_pair("ABab?.^{}_cdFGETG123)(", TextConfig()));
 			texts.push_back(std::make_pair("Hello, World!", TextConfig()));
 			texts.push_back(std::make_pair("\u0410\u0411\u0412\u041F", TextConfig()));
-			texts.push_back(std::make_pair("Text with unsupported glyphs \u1E30\u1E31 is coming", TextConfig()));
+			texts.push_back(std::make_pair("Unsupported glyphs \u1E30\u1E31 are coming", TextConfig()));
 			texts.push_back(std::make_pair("This is first line\nSecond gg line\nThird G line", TextConfig()));
 			texts[0].second.applyBorder = true;
 			texts[1].second.backgroundColor.a = 1;
@@ -65,27 +65,44 @@ namespace OpenVulkano
 			const int N = texts.size();
 			auto& resourceLoader = ResourceLoader::GetInstance();
 			const std::string fontPath = resourceLoader.GetResourcePath("Roboto-Regular.ttf");
-			m_nodesPool.resize(N);
+			m_nodesPool.resize(N * 2);
-			m_drawablesPool.resize(N);
+			m_drawablesPool.resize(N * 2);
 		
 #ifdef MSDFGEN_AVAILABLE
-			msdf_atlas::Charset charset = MsdfFontAtlasGenerator::LoadAllGlyphs(fontPath);
+			msdf_atlas::Charset charset = SdfFontAtlasGenerator::LoadAllGlyphs(fontPath);
-			//msdf_atlas::Charset charset = msdf_atlas::Charset::ASCII;
-			//for (msdf_atlas::unicode_t c = 0x0410; c <= 0x041F; c++)
-			//{
-			//	// some unicode values
-			//	charset.add(c);
-			//}
 			m_atlasGenerator.GenerateAtlas(fontPath, charset);
+			m_msdfAtlasGenerator.GenerateAtlas(fontPath, charset);
+#else 
+			auto sdfMetadataInfo = resourceLoader.GetResource("sdf_atlas_packed");
+			auto msdfMetadataInfo = resourceLoader.GetResource("msdf_atlas_packed.png");
 #endif
 
-			for (int i = 0; i < texts.size(); i++)
+			for (int i = 0; i < texts.size() * 2; i++)
 			{
+				int textIdx = i % texts.size();
+				TextDrawable* t = nullptr;
 #ifdef MSDFGEN_AVAILABLE
-				TextDrawable* t = new TextDrawable(&m_atlasGenerator, texts[i].second);
+				if (i < texts.size()) 
+				{ 
+					t = new TextDrawable(&m_atlasGenerator, texts[textIdx].second); 
+					t->SetShader(&TextDrawable::GetSdfDefaultShader());
+				}
+				else
+				{ 
+					t = new TextDrawable(&m_msdfAtlasGenerator, texts[textIdx].second); 
+					t->SetShader(&TextDrawable::GetMsdfDefaultShader());
+				}
 #else
-				auto metadataInfo = resourceLoader.GetResource("full_atlas_metadata_packed");
+				if (i < texts.size()) 
-				TextDrawable* t = new TextDrawable(metadataInfo, texts[i].second);
+				{
+					t = new TextDrawable(sdfMetadataInfo, texts[textIdx].second); 
+					t->SetShader(&TextDrawable::GetSdfDefaultShader());
+				}
+				else 
+				{
+					t = new TextDrawable(msdfMetadataInfo, texts[textIdx].second); 
+					t->SetShader(&TextDrawable::GetMsdfDefaultShader());
+				}
 				// OR use separate texture + metadata file
 				//auto metadataInfo = resourceLoader.GetResource("atlas_metadata");
 				//auto data = resourceLoader.GetResource("roboto-regular-atlas.png");
@@ -98,17 +115,17 @@ namespace OpenVulkano
 				//tex.size = image->data.Size(); // 1 channel
 				//TextDrawable* t = new TextDrawable(metadataInfo, &tex, texts[i].second);
 #endif // MSDFGEN_AVAILABLE 
-				t->GenerateText(texts[i].first);
+				t->GenerateText(texts[textIdx].first);
 				m_drawablesPool[i] = t;
 				m_nodesPool[i].Init();
-				m_nodesPool[i].SetMatrix(Math::Utils::translate(glm::mat4x4(1.f), Vector3f(-5, 2 - i * 2, 0)));
+				m_nodesPool[i].SetMatrix(Math::Utils::translate(glm::mat4x4(1.f), Vector3f((i < texts.size() ? -5 : 15), 2 - textIdx * 2, 0)));
 				m_nodesPool[i].AddDrawable(m_drawablesPool[i]);
 				m_scene.GetRoot()->AddChild(&m_nodesPool[i]);
 			}
 			GetGraphicsAppManager()->GetRenderer()->SetScene(&m_scene);
 			m_camController.Init(&m_cam);
 			m_camController.SetDefaultKeybindings();
-			m_camController.SetPosition({ 0, 0, 10 });
+			m_camController.SetPosition({ 10, 0, 15 });
 			m_camController.SetBoostFactor(5);
 			
 			std::shared_ptr<OpenVulkano::Scene::UI::PerformanceInfo> m_perfInfo =
@@ -126,6 +143,7 @@ namespace OpenVulkano
 		{
 			for (SimpleDrawable* d: m_drawablesPool)
 			{
+				d->Close();
 				delete d;
 			}
 		}
@@ -135,7 +153,8 @@ namespace OpenVulkano
 		PerspectiveCamera m_cam;
 		OpenVulkano::FreeCamCameraController m_camController;
 #ifdef MSDFGEN_AVAILABLE
-		MsdfFontAtlasGenerator m_atlasGenerator;
+		SdfFontAtlasGenerator m_atlasGenerator;
+		MsdfFontAtlasGenerator m_msdfAtlasGenerator;
 #endif
 		std::vector<SimpleDrawable*> m_drawablesPool;
 		std::vector<Node> m_nodesPool;

openVulkanoCpp/Image/ImageLoader.cpp

@@ -25,18 +25,15 @@ namespace OpenVulkano::Image
 			"Stbi load image channels mismatch. Desired channels = {}, actual amount of channels in image = {}",
 			desiredChannels, channels);
 		}
-		result.data = OpenVulkano::Array<uint8_t>(cols * rows * channels);
 		switch (channels)
 		{
 			case 1:
 				result.dataFormat = OpenVulkano::DataFormat::R8_UNORM;
 				break;
 			case 2:
-				result.dataFormat = OpenVulkano::DataFormat::R8G8_UNORM;
+				result.dataFormat = OpenVulkano::DataFormat::R8G8_UNORM; 
 				break;
 			case 3:
-				result.dataFormat = OpenVulkano::DataFormat::R8G8B8_UNORM;
-				break;
 			case 4:
 				result.dataFormat = OpenVulkano::DataFormat::R8G8B8A8_UNORM;
 				break;
@@ -44,7 +41,22 @@ namespace OpenVulkano::Image
 		result.resolution.x = cols;
 		result.resolution.y = rows;
 		result.resolution.z = 1;
-		std::memcpy(result.data.Data(), pixelData, result.data.Size());
+		if (channels == 3)
+		{ 
+			result.data = OpenVulkano::Array<uint8_t>(cols * rows * 4);
+			for (size_t srcPos = 0, dstPos = 0; srcPos < cols * rows * 3; srcPos += 3, dstPos += 4)
+			{
+				result.data[dstPos] = pixelData[srcPos];
+				result.data[dstPos + 1] = pixelData[srcPos + 1];
+				result.data[dstPos + 2] = pixelData[srcPos + 2];
+				result.data[dstPos + 3] = 255;
+			}
+		}
+		else
+		{ 
+			result.data = OpenVulkano::Array<uint8_t>(cols * rows * channels);
+			std::memcpy(result.data.Data(), pixelData, result.data.Size());
+		}
 		stbi_image_free(pixelData);
 		return std::make_unique<Image>(std::move(result));
 	}

openVulkanoCpp/Scene/FontAtlasGenerator.cpp

@@ -6,7 +6,7 @@
 
 #if __has_include("msdfgen.h")
 
-#include "MsdfFontAtlasGenerator.hpp"
+#include "FontAtlasGenerator.hpp"
 #include "Base/Logger.hpp"
 #include "Scene/AtlasMetadata.hpp"
 #include <msdfgen.h>
@@ -24,8 +24,12 @@ namespace OpenVulkano::Scene
 {
 	using namespace msdfgen;
 	using namespace msdf_atlas;
+	
+	FontAtlasGeneratorConfig FontAtlasGeneratorConfig::sdfDefaultConfig = { 42, 1.0, 5 };
+	FontAtlasGeneratorConfig FontAtlasGeneratorConfig::msdfDefaultConfig = { 32, 1.0, 3 };
 
-	Charset MsdfFontAtlasGenerator::LoadAllGlyphs(const std::variant<std::string, Array<char>>& data) 
+	template<int Channels>
+	Charset FontAtlasGenerator<Channels>::LoadAllGlyphs(const std::variant<std::string, Array<char>>& data) 
 	{ 
 		FT_Library library;
 		auto error = FT_Init_FreeType(&library);
@@ -58,7 +62,8 @@ namespace OpenVulkano::Scene
 		return s;
 	}
 
-	void MsdfFontAtlasGenerator::GenerateAtlas(const std::string& fontFile, const std::set<uint32_t>& charset,
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::GenerateAtlas(const std::string& fontFile, const std::set<uint32_t>& charset,
 	                                           const std::optional<std::string>& pngOutput)
 	{
 		FreetypeHandle* ft;
@@ -69,7 +74,15 @@ namespace OpenVulkano::Scene
 		Generate(ft, font, s, pngOutput);
 	}
 
-	void MsdfFontAtlasGenerator::GenerateAtlas(const Array<char>& fontData, int length,
+	template<int Channels>
+	FontAtlasGenerator<Channels>::FontAtlasGenerator() 
+	{ 
+		if constexpr (Channels == 1) m_config = FontAtlasGeneratorConfig::sdfDefaultConfig;
+		else m_config = FontAtlasGeneratorConfig::msdfDefaultConfig;
+	}
+
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::GenerateAtlas(const Array<char>& fontData, int length,
 	                                           const std::set<uint32_t>& charset,
 	                                           const std::optional<std::string>& pngOutput)
 	{
@@ -81,7 +94,8 @@ namespace OpenVulkano::Scene
 		Generate(ft, font, s, pngOutput);
 	}
 
-	void MsdfFontAtlasGenerator::GenerateAtlas(const std::string& fontFile, const Charset& charset,
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::GenerateAtlas(const std::string& fontFile, const Charset& charset,
 	                                           const std::optional<std::string>& pngOutput)
 	{
 		// TODO: dynamic atlas and add only those symbols which are not present yet in current atlas
@@ -91,7 +105,9 @@ namespace OpenVulkano::Scene
 		Generate(ft, font, charset, pngOutput);
 	}
 
-	void MsdfFontAtlasGenerator::GenerateAtlas(const msdfgen::byte* fontData, int length, const Charset& charset,
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::GenerateAtlas(const msdfgen::byte* fontData, int length,
+	                                                     const Charset& charset,
 	                                           const std::optional<std::string>& pngOutput)
 	{
 		FreetypeHandle* ft;
@@ -100,7 +116,9 @@ namespace OpenVulkano::Scene
 		Generate(ft, font, charset, pngOutput);
 	}
 
-	void MsdfFontAtlasGenerator::SaveAtlasMetadataInfo(const std::string& outputFile, bool packIntoSingleFile) const 
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::SaveAtlasMetadataInfo(const std::string& outputFile,
+	                                                             bool packIntoSingleFile) const 
 	{
 		if (m_symbols.empty()) 
 		{
@@ -111,17 +129,9 @@ namespace OpenVulkano::Scene
 		uint32_t packedFlag = packIntoSingleFile;
 		if (packIntoSingleFile) 
 		{
-			size_t ext = outputFile.find_last_of('.');
+			std::filesystem::path fPath(fileName);
-			if (ext == std::string::npos)
+			fileName = (fPath.parent_path() / fPath.stem()).string() + "_packed.png";
-			{
+			SavePng(fileName);
-				fileName += "_packed";
-			}
-			else
-			{
-				fileName.insert(ext - 1, "_packed");
-			}
-			const BitmapConstRef<byte, 1>& storage = m_generator.atlasStorage();
-			SavePng(m_generator.atlasStorage(), fileName, 1);
 		}
 		std::fstream fs(fileName.c_str(), std::ios_base::out | std::ios_base::binary | (packedFlag ? std::ios_base::app : std::ios_base::trunc));
 		fs.write(reinterpret_cast<const char*>(&m_meta), sizeof(AtlasMetadata));
@@ -137,7 +147,9 @@ namespace OpenVulkano::Scene
 		fs.write(reinterpret_cast<const char*>(&packedFlag), sizeof(uint32_t));
 	}
 
-	void MsdfFontAtlasGenerator::InitFreetypeFromFile(FreetypeHandle*& ft, FontHandle*& font, const std::string& fontFile)
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::InitFreetypeFromFile(FreetypeHandle*& ft, FontHandle*& font,
+	                                                            const std::string& fontFile)
 	{
 		ft = initializeFreetype();
 		if (!ft) { throw std::runtime_error("Failed to initialize freetype"); }
@@ -150,7 +162,8 @@ namespace OpenVulkano::Scene
 		}
 	}
 
-	void MsdfFontAtlasGenerator::InitFreetypeFromBuffer(FreetypeHandle*& ft, FontHandle*& font,
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::InitFreetypeFromBuffer(FreetypeHandle*& ft, FontHandle*& font,
 	                                                    const msdfgen::byte* fontData, int length)
 	{
 		ft = initializeFreetype();
@@ -164,7 +177,8 @@ namespace OpenVulkano::Scene
 		}
 	}
 
-	void MsdfFontAtlasGenerator::Generate(FreetypeHandle* ft, FontHandle* font, const Charset& chset,
+	template<int Channels>
+	void FontAtlasGenerator<Channels>::Generate(FreetypeHandle* ft, FontHandle* font, const Charset& chset,
 	                                      const std::optional<std::string>& pngOutput)
 	{
 		m_symbols.clear();
@@ -173,6 +187,13 @@ namespace OpenVulkano::Scene
 		FontGeometry fontGeometry(&glyphsGeometry);
 		fontGeometry.loadCharset(font, 1, chset);
 
+		if constexpr (Channels == 3)
+		{
+			const double maxCornerAngle = 3.0;
+			for (GlyphGeometry& glyph: glyphsGeometry)
+				glyph.edgeColoring(&msdfgen::edgeColoringByDistance, maxCornerAngle, 0);
+		}
+
 		TightAtlasPacker packer;
 		packer.setDimensionsConstraint(DimensionsConstraint::SQUARE);
 		int width, height;
@@ -187,18 +208,38 @@ namespace OpenVulkano::Scene
 		packer.pack(glyphsGeometry.data(), glyphsGeometry.size());
 		packer.getDimensions(width, height);
 
-		m_generator.resize(width, height);
+		Generator generator;
+		generator.resize(width, height);
 		GeneratorAttributes attributes;
-		m_generator.setAttributes(attributes);
+		generator.setAttributes(attributes);
-		m_generator.setThreadCount(4);
+		generator.setThreadCount(4);
-		m_generator.generate(glyphsGeometry.data(), glyphsGeometry.size());
+		generator.generate(glyphsGeometry.data(), glyphsGeometry.size());
 
 		int idx = 0;
-		const BitmapConstRef<byte, 1>& storage = m_generator.atlasStorage();
+		if constexpr (Channels == 3)
-		m_atlasTex.resolution = Math::Vector3ui(storage.width, storage.height, 1);
+		{
-		m_atlasTex.textureBuffer = (msdfgen::byte*) storage.pixels;
+			// store RGB as RGBA
-		m_atlasTex.format = OpenVulkano::DataFormat::R8_UNORM;
+			const BitmapConstRef<msdfgen::byte, 3> storage = generator.atlasStorage();
-		m_atlasTex.size = storage.width * storage.height * 1; // 1 channel
+			msdfgen::Bitmap<msdfgen::byte, 4> bitmap(width, height);
+			msdfgen::byte* data = static_cast<msdfgen::byte*>(bitmap);
+			for (size_t srcPos = 0, dstPos = 0; srcPos < width * height * 3; srcPos += 3, dstPos += 4)
+			{
+				data[dstPos] = storage.pixels[srcPos];
+				data[dstPos + 1] = storage.pixels[srcPos + 1];
+				data[dstPos + 2] = storage.pixels[srcPos + 2];
+				data[dstPos + 3] = 255;
+			}
+			m_atlasStorage = std::move(bitmap);
+		}
+		else 
+		{ 
+			m_atlasStorage = generator.atlasStorage();
+		}
+
+		m_atlasTex.resolution = Math::Vector3ui(m_atlasStorage.width(), m_atlasStorage.height(), 1);
+		m_atlasTex.textureBuffer = (msdfgen::byte*) m_atlasStorage;
+		m_atlasTex.format = (channelsCount == 1 ? OpenVulkano::DataFormat::R8_UNORM : OpenVulkano::DataFormat::R8G8B8A8_UNORM);
+		m_atlasTex.size = m_atlasStorage.width() * m_atlasStorage.height() * channelsCount;
 
 		m_meta.lineHeight = fontGeometry.getMetrics().lineHeight;
 
@@ -210,7 +251,7 @@ namespace OpenVulkano::Scene
 		for (const auto& glyph: glyphsGeometry)
 		{
 			GlyphInfo& info = m_symbols[glyph.getCodepoint()];
-			const GlyphBox& glyphBox = m_generator.getLayout()[idx++];
+			const GlyphBox& glyphBox = generator.getLayout()[idx++];
 
 			Bbox glyphBaselineBbox, glyphAtlasBbox;
 			glyph.getQuadPlaneBounds(glyphBaselineBbox.l, glyphBaselineBbox.b, glyphBaselineBbox.r,
@@ -252,25 +293,27 @@ namespace OpenVulkano::Scene
 			info.advance = glyphBox.advance;
 		}
 
-		if (pngOutput && !pngOutput->empty()) { SavePng(storage, pngOutput.value(), 1); }
+		if (pngOutput && !pngOutput->empty()) { SavePng(pngOutput.value()); }
 		destroyFont(font);
 		deinitializeFreetype(ft);
 	}
 
-	void MsdfFontAtlasGenerator::SavePng(const BitmapConstRef<msdfgen::byte, 1>& storage, const std::string& output,
+	template<int Channels>
-	                                     int channels) const
+	void FontAtlasGenerator<Channels>::SavePng(const std::string& output) const
 	{
 		stbi_flip_vertically_on_write(1);
 		if (std::filesystem::path(output).extension() == ".png")
 		{
-			stbi_write_png(output.c_str(), storage.width, storage.height, channels, storage.pixels,
+			stbi_write_png(output.c_str(), m_atlasStorage.width(), m_atlasStorage.height(), channelsCount,
-			               channels * storage.width);
+			               m_atlasStorage.operator const msdfgen::byte*(), channelsCount * m_atlasStorage.width());
 		}
 		else
 		{
-			stbi_write_png((output + ".png").c_str(), storage.width, storage.height, channels, storage.pixels,
+			stbi_write_png((output + ".png").c_str(), m_atlasStorage.width(), m_atlasStorage.height(), channelsCount,
-			               channels * storage.width);
+			               m_atlasStorage.operator const msdfgen::byte*(), channelsCount * m_atlasStorage.width());
 		}
 	}
+	template class FontAtlasGenerator<1>;
+	template class FontAtlasGenerator<3>;
 }
 #endif

openVulkanoCpp/Scene/FontAtlasGenerator.hpp

@@ -6,27 +6,77 @@
 
 #pragma once
 
+#if __has_include("msdfgen.h")
+
 #include "Scene/AtlasMetadata.hpp"
+#include "IFontAtlasGenerator.hpp"
 #include "Scene/Texture.hpp"
+#include <msdfgen.h>
+#include <msdf-atlas-gen/msdf-atlas-gen.h>
 #include <string>
 #include <optional>
 #include <map>
 #include <variant>
-#include <set>
+#define MSDFGEN_AVAILABLE 1
-#include <memory>
 
 namespace OpenVulkano::Scene
 {
-	class FontAtlasGenerator
+
+	struct FontAtlasGeneratorConfig
 	{
-	public:
+		static FontAtlasGeneratorConfig sdfDefaultConfig;
-		virtual void GenerateAtlas(const std::string& fontFile, const std::set<uint32_t>& charset,
+		static FontAtlasGeneratorConfig msdfDefaultConfig;
-		                           const std::optional<std::string>& pngOutput = std::nullopt) = 0;
+		int glyphSize;
-		virtual void GenerateAtlas(const Array<char>& fontData, int length, const std::set<uint32_t>& charset,
+		double miterLimit;
-		                           const std::optional<std::string>& pngOutput = std::nullopt) = 0;
+		msdfgen::Range pixelRange;
-		virtual void SaveAtlasMetadataInfo(const std::string& outputFile, bool packIntoSingleFile = true) const = 0;
-		virtual const Texture& GetAtlas() const = 0;
-		virtual std::map<uint32_t, GlyphInfo>& GetGlyphsInfo() = 0;
-		virtual AtlasMetadata& GetAtlasMetadata() = 0;
 	};
+
+	template<int Channels>
+	class FontAtlasGenerator : public IFontAtlasGenerator
+	{
+	private:
+		using SdfGenerator = msdf_atlas::ImmediateAtlasGenerator<float, 1, msdf_atlas::sdfGenerator,
+		                                                         msdf_atlas::BitmapAtlasStorage<msdfgen::byte, 1>>;
+		using MsdfGenerator = msdf_atlas::ImmediateAtlasGenerator<float, 3, msdf_atlas::msdfGenerator,
+		                                                          msdf_atlas::BitmapAtlasStorage<msdfgen::byte, 3>>;
+	public:
+		using Generator = std::conditional<Channels == 1, SdfGenerator, MsdfGenerator>::type;
+		using AtlasData = std::conditional<Channels == 1, msdfgen::Bitmap<msdfgen::byte, 1>, msdfgen::Bitmap<msdfgen::byte, 4>>::type;
+		using Config = FontAtlasGeneratorConfig;
+		static constexpr int channelsCount = (Channels == 1 ? 1 : 4);
+		static msdf_atlas::Charset LoadAllGlyphs(const std::variant<std::string, Array<char>>& data);
+		FontAtlasGenerator();
+		void GenerateAtlas(const std::string& fontFile, const std::set<uint32_t>& charset,
+		                   const std::optional<std::string>& pngOutput = std::nullopt) override;
+		void GenerateAtlas(const Array<char>& fontData, int length, const std::set<uint32_t>& charset,
+		                   const std::optional<std::string>& pngOutput = std::nullopt) override;
+		void GenerateAtlas(const std::string& fontFile, const msdf_atlas::Charset& charset = msdf_atlas::Charset::ASCII,
+		                   const std::optional<std::string>& pngOutput = std::nullopt);
+		void GenerateAtlas(const msdfgen::byte* fontData, int length,
+		                   const msdf_atlas::Charset& charset = msdf_atlas::Charset::ASCII,
+		                   const std::optional<std::string>& pngOutput = std::nullopt);
+		void SaveAtlasMetadataInfo(const std::string& outputFile, bool packIntoSingleFile = true) const override;
+		void SetGeneratorConfig(const Config& config) { m_config = config; }
+		const Texture& GetAtlas() const override { return m_atlasTex; }
+		std::map<uint32_t, GlyphInfo>& GetGlyphsInfo() override { return m_symbols; }
+		AtlasMetadata& GetAtlasMetadata() override { return m_meta; }
+		Config& GetGeneratorConfig() { return m_config; }
+	private:
+		void InitFreetypeFromFile(msdfgen::FreetypeHandle*& ft, msdfgen::FontHandle*& font, const std::string& file);
+		void InitFreetypeFromBuffer(msdfgen::FreetypeHandle*& ft, msdfgen::FontHandle*& font,
+		                            const msdfgen::byte* fontData, int length);
+		void Generate(msdfgen::FreetypeHandle* ft, msdfgen::FontHandle* font, const msdf_atlas::Charset& chset,
+		              const std::optional<std::string>& pngOutput);
+		void SavePng(const std::string& output) const;
+
+	private:
+		Texture m_atlasTex;
+		AtlasMetadata m_meta;
+		Config m_config;
+		std::map<uint32_t, GlyphInfo> m_symbols;
+		AtlasData m_atlasStorage;
+	};
+	using SdfFontAtlasGenerator = FontAtlasGenerator<1>;
+	using MsdfFontAtlasGenerator = FontAtlasGenerator<3>;
 }
+#endif

openVulkanoCpp/Scene/IFontAtlasGenerator.hpp

@@ -0,0 +1,32 @@
+/*
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at https://mozilla.org/MPL/2.0/.
+ */
+
+#pragma once
+
+#include "Scene/AtlasMetadata.hpp"
+#include "Scene/Texture.hpp"
+#include <string>
+#include <optional>
+#include <map>
+#include <variant>
+#include <set>
+#include <memory>
+
+namespace OpenVulkano::Scene
+{
+	class IFontAtlasGenerator
+	{
+	public:
+		virtual void GenerateAtlas(const std::string& fontFile, const std::set<uint32_t>& charset,
+		                           const std::optional<std::string>& pngOutput = std::nullopt) = 0;
+		virtual void GenerateAtlas(const Array<char>& fontData, int length, const std::set<uint32_t>& charset,
+		                           const std::optional<std::string>& pngOutput = std::nullopt) = 0;
+		virtual void SaveAtlasMetadataInfo(const std::string& outputFile, bool packIntoSingleFile = true) const = 0;
+		virtual const Texture& GetAtlas() const = 0;
+		virtual std::map<uint32_t, GlyphInfo>& GetGlyphsInfo() = 0;
+		virtual AtlasMetadata& GetAtlasMetadata() = 0;
+	};
+}

openVulkanoCpp/Scene/MsdfFontAtlasGenerator.hpp

@@ -1,72 +0,0 @@
-/*
- * This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at https://mozilla.org/MPL/2.0/.
- */
-
-#pragma once
-
-#if __has_include("msdfgen.h")
-
-#include "Scene/AtlasMetadata.hpp"
-#include "FontAtlasGenerator.hpp"
-#include "Scene/Texture.hpp"
-#include <msdfgen.h>
-#include <msdf-atlas-gen/msdf-atlas-gen.h>
-#include <string>
-#include <optional>
-#include <map>
-#include <variant>
-#define MSDFGEN_AVAILABLE 1
-
-namespace OpenVulkano::Scene
-{
-
-	struct MsdfFontAtlasGeneratorConfig
-	{
-		int glyphSize = 42;
-		double miterLimit = 1.0;
-		msdfgen::Range pixelRange = 5;
-	};
-
-	class MsdfFontAtlasGenerator : public FontAtlasGenerator
-	{
-	public:
-		using SdfGenerator = msdf_atlas::ImmediateAtlasGenerator<float, 1, msdf_atlas::sdfGenerator,
-		                                                         msdf_atlas::BitmapAtlasStorage<msdfgen::byte, 1>>;
-		using Config = MsdfFontAtlasGeneratorConfig;
-		static msdf_atlas::Charset LoadAllGlyphs(const std::variant<std::string, Array<char>>& data);
-		void GenerateAtlas(const std::string& fontFile, const std::set<uint32_t>& charset,
-		                   const std::optional<std::string>& pngOutput = std::nullopt) override;
-		void GenerateAtlas(const Array<char>& fontData, int length, const std::set<uint32_t>& charset,
-		                   const std::optional<std::string>& pngOutput = std::nullopt) override;
-		void GenerateAtlas(const std::string& fontFile, const msdf_atlas::Charset& charset = msdf_atlas::Charset::ASCII,
-		                   const std::optional<std::string>& pngOutput = std::nullopt);
-		void GenerateAtlas(const msdfgen::byte* fontData, int length,
-		                   const msdf_atlas::Charset& charset = msdf_atlas::Charset::ASCII,
-		                   const std::optional<std::string>& pngOutput = std::nullopt);
-		void SaveAtlasMetadataInfo(const std::string& outputFile, bool packIntoSingleFile = true) const override;
-		void SetGeneratorConfig(const Config& config) { m_config = config; }
-		const Texture& GetAtlas() const override { return m_atlasTex; }
-		std::map<uint32_t, GlyphInfo>& GetGlyphsInfo() override { return m_symbols; }
-		AtlasMetadata& GetAtlasMetadata() override { return m_meta; }
-		SdfGenerator& GetFontAtlasGenerator() { return m_generator; }
-		Config& GetGeneratorConfig() { return m_config; }
-	private:
-		void InitFreetypeFromFile(msdfgen::FreetypeHandle*& ft, msdfgen::FontHandle*& font, const std::string& file);
-		void InitFreetypeFromBuffer(msdfgen::FreetypeHandle*& ft, msdfgen::FontHandle*& font,
-		                            const msdfgen::byte* fontData, int length);
-		void Generate(msdfgen::FreetypeHandle* ft, msdfgen::FontHandle* font, const msdf_atlas::Charset& chset,
-		              const std::optional<std::string>& pngOutput);
-		void SavePng(const msdfgen::BitmapConstRef<msdfgen::byte, 1>& storage, const std::string& output,
-		             int channels) const;
-
-	private:
-		SdfGenerator m_generator;
-		Texture m_atlasTex;
-		AtlasMetadata m_meta;
-		Config m_config;
-		std::map<uint32_t, GlyphInfo> m_symbols;
-	};
-}
-#endif

openVulkanoCpp/Scene/TextDrawable.cpp

@@ -9,7 +9,7 @@
 #include "Scene/Material.hpp"
 #include "Scene/Vertex.hpp"
 #include "Scene/UniformBuffer.hpp"
-#include "Scene/FontAtlasGenerator.hpp"
+#include "Scene/IFontAtlasGenerator.hpp"
 #include "Base/Logger.hpp"
 #include "Host/ResourceLoader.hpp"
 #include "Image/ImageLoader.hpp"
@@ -20,22 +20,40 @@
 
 namespace OpenVulkano::Scene
 {
-	Shader& TextDrawable::GetDefaultShader()
+	Shader& TextDrawable::GetSdfDefaultShader()
 	{
 		static bool once = true;
-		static Shader textDefaultShader;
+		static Shader sdfDefaultShader;
 		if (once)
 		{
-			textDefaultShader.AddShaderProgram(OpenVulkano::ShaderProgramType::VERTEX, "Shader/text");
+			sdfDefaultShader.AddShaderProgram(OpenVulkano::ShaderProgramType::VERTEX, "Shader/text");
-			textDefaultShader.AddShaderProgram(OpenVulkano::ShaderProgramType::FRAGMENT, "Shader/text");
+			sdfDefaultShader.AddShaderProgram(OpenVulkano::ShaderProgramType::FRAGMENT, "Shader/text");
-			textDefaultShader.AddVertexInputDescription(OpenVulkano::Vertex::GetVertexInputDescription());
+			sdfDefaultShader.AddVertexInputDescription(OpenVulkano::Vertex::GetVertexInputDescription());
-			textDefaultShader.AddDescriptorSetLayoutBinding(Texture::DESCRIPTOR_SET_LAYOUT_BINDING);
+			sdfDefaultShader.AddDescriptorSetLayoutBinding(Texture::DESCRIPTOR_SET_LAYOUT_BINDING);
-			textDefaultShader.AddDescriptorSetLayoutBinding(UniformBuffer::DESCRIPTOR_SET_LAYOUT_BINDING);
+			sdfDefaultShader.AddDescriptorSetLayoutBinding(UniformBuffer::DESCRIPTOR_SET_LAYOUT_BINDING);
-			textDefaultShader.alphaBlend = true;
+			sdfDefaultShader.alphaBlend = true;
-			textDefaultShader.cullMode = CullMode::NONE;
+			sdfDefaultShader.cullMode = CullMode::NONE;
 			once = false;
 		}
-		return textDefaultShader;
+		return sdfDefaultShader;
+	}
+
+	Shader& TextDrawable::GetMsdfDefaultShader()
+	{
+		static bool once = true;
+		static Shader msdfDefaultShader;
+		if (once)
+		{
+			msdfDefaultShader.AddShaderProgram(OpenVulkano::ShaderProgramType::VERTEX, "Shader/text");
+			msdfDefaultShader.AddShaderProgram(OpenVulkano::ShaderProgramType::FRAGMENT, "Shader/msdfText");
+			msdfDefaultShader.AddVertexInputDescription(OpenVulkano::Vertex::GetVertexInputDescription());
+			msdfDefaultShader.AddDescriptorSetLayoutBinding(Texture::DESCRIPTOR_SET_LAYOUT_BINDING);
+			msdfDefaultShader.AddDescriptorSetLayoutBinding(UniformBuffer::DESCRIPTOR_SET_LAYOUT_BINDING);
+			msdfDefaultShader.alphaBlend = true;
+			msdfDefaultShader.cullMode = CullMode::NONE;
+			once = false;
+		}
+		return msdfDefaultShader;
 	}
 
 	TextDrawable::TextDrawable(const Array<char>& atlasMetadata, const TextConfig& config)
@@ -63,12 +81,13 @@ namespace OpenVulkano::Scene
 		uint64_t offsetToMetadata = atlasMetadata.Size() - metadataBytes - sizeof(uint32_t) - sizeof(uint64_t);
 		if (isPacked)
 		{
-			m_material.texture = new Texture();
+			m_texture = Texture();
+			m_material.texture = &m_texture.value();
 			m_img = Image::IImageLoader::loadData((const uint8_t*) atlasMetadata.Data(),
 			                                      offsetToMetadata);
 			m_material.texture->format = m_img->dataFormat;
 			m_material.texture->resolution = m_img->resolution;
-			m_material.texture->size = m_img->data.Size(); // 1 channel
+			m_material.texture->size = m_img->data.Size();
 			m_material.texture->textureBuffer = m_img->data.Data();
 		}
 		else
@@ -110,7 +129,7 @@ namespace OpenVulkano::Scene
 		m_uniBuffer.Init(sizeof(TextConfig), &m_cfg, 3);
 	}
 
-	TextDrawable::TextDrawable(FontAtlasGenerator* fontAtlasGenerator, const TextConfig& config)
+	TextDrawable::TextDrawable(IFontAtlasGenerator* fontAtlasGenerator, const TextConfig& config)
 	{
 		if (!fontAtlasGenerator) { throw std::runtime_error("FontAtlasGenerator is nullptr"); }
 		if (fontAtlasGenerator->GetGlyphsInfo().empty()) { throw std::runtime_error("Glyphs are not loaded"); }
@@ -237,7 +256,7 @@ namespace OpenVulkano::Scene
 		SimpleDrawable::Init(m_shader, &m_geometry, &m_material, &m_uniBuffer);
 	}
 
-	void TextDrawable::SetFontAtlasGenerator(FontAtlasGenerator* fontAtlasGenerator) 
+	void TextDrawable::SetFontAtlasGenerator(IFontAtlasGenerator* fontAtlasGenerator) 
 	{
 		if (!fontAtlasGenerator || fontAtlasGenerator->GetGlyphsInfo().empty())
 		{

openVulkanoCpp/Scene/TextDrawable.hpp

@@ -14,11 +14,12 @@
 #include "AtlasMetadata.hpp"
 #include "Image/Image.hpp"
 #include <map>
+#include <optional>
 
 namespace OpenVulkano::Scene
 {
 
-	class FontAtlasGenerator;
+	class IFontAtlasGenerator;
 
 	struct TextConfig
 	{
@@ -35,20 +36,21 @@ namespace OpenVulkano::Scene
 	class TextDrawable : public SimpleDrawable
 	{
 	public:
-		static Shader& GetDefaultShader();
+		static Shader& GetSdfDefaultShader();
+		static Shader& GetMsdfDefaultShader();
 		TextDrawable(const Array<char>& atlasMetadata, const TextConfig& config = TextConfig());
 		TextDrawable(const std::string& atlasMetadataFile, const TextConfig& config = TextConfig());
 		TextDrawable(const std::string& atlasMetadataFile, Texture* atlasTex, const TextConfig& config = TextConfig());
 		TextDrawable(const Array<char>& atlasMetadata, Texture* atlasTex, const TextConfig& config = TextConfig());
 		TextDrawable(const std::map<uint32_t, GlyphInfo>& glyphData, Texture* atlasTex, const TextConfig& config = TextConfig());
-		TextDrawable(FontAtlasGenerator* fontAtlasGenerator, const TextConfig& config = TextConfig());
+		TextDrawable(IFontAtlasGenerator* fontAtlasGenerator, const TextConfig& config = TextConfig());
 		void GenerateText(const std::string& text, const Math::Vector3f& pos = Math::Vector3f(0.f)); 
 		void SetConfig(const TextConfig& cfg) { m_cfg = cfg; }
 		void SetShader(Shader* shader) { m_shader = shader; }
 		TextConfig& GetConfig() { return m_cfg; }
 		Shader* GetShader() { return m_shader; }
-		void SetFontAtlasGenerator(FontAtlasGenerator* fontAtlasGenerator);
+		void SetFontAtlasGenerator(IFontAtlasGenerator* fontAtlasGenerator);
-		FontAtlasGenerator* GetFontAtlasGenerator() { return m_fontAtlasGenerator; }
+		IFontAtlasGenerator* GetFontAtlasGenerator() { return m_fontAtlasGenerator; }
 	private:
 		Geometry m_geometry;
 		Material m_material;
@@ -56,8 +58,9 @@ namespace OpenVulkano::Scene
 		std::map<uint32_t, GlyphInfo> m_glyphs;
 		AtlasMetadata m_meta;
 		std::unique_ptr<Image::Image> m_img;
-		FontAtlasGenerator* m_fontAtlasGenerator = nullptr;
+		std::optional<Texture> m_texture;
-		Shader* m_shader = &GetDefaultShader();
+		IFontAtlasGenerator* m_fontAtlasGenerator = nullptr;
+		Shader* m_shader = nullptr;
 		TextConfig m_cfg;
 	};
 }

openVulkanoCpp/Shader/msdfText.frag

@@ -0,0 +1,47 @@
+#version 450
+
+layout(location = 0) in vec2 texCoord;
+
+layout(location = 0) out vec4 outColor;
+
+layout(set = 2, binding = 0) uniform sampler2D texSampler;
+
+layout(set = 3, binding = 0) uniform TextConfig
+{
+    vec4 textColor;
+    vec4 borderColor;
+    vec4 backgroundColor;
+    float threshold;
+	float borderSize;
+	float smoothing;
+	bool applyBorder;
+} textConfig;
+
+float median(float r, float g, float b) {
+    return max(min(r, g), min(max(r, g), b));
+}
+
+// this parameter should be same as FontAtlasGeneratorConfig::pixelRange
+const float pxRange = 3;
+
+float screenPxRange() {
+    vec2 unitRange = vec2(pxRange)/vec2(textureSize(texSampler, 0));
+    vec2 screenTexSize = vec2(1.0)/fwidth(texCoord);
+    return max(0.5*dot(unitRange, screenTexSize), 1.0);
+}
+
+void main()
+{
+    vec3 msd = texture(texSampler, texCoord).rgb;
+    float sd = median(msd.r, msd.g, msd.b);
+    float screenPxDistance = screenPxRange()*(sd - 0.5);
+    float opacity = clamp(screenPxDistance + 0.5, 0.0, 1.0);
+    if (textConfig.backgroundColor.a != 0)
+    {
+       outColor = mix(textConfig.backgroundColor, textConfig.textColor, opacity);
+    }
+    else
+    {
+       outColor = vec4(vec3(textConfig.textColor), opacity);
+    }
+}