Fixed bug where images wouldn't be properly exported

Author: Sekqies

src/graphics/ui.cpp

@@ -342,9 +342,6 @@ void render_and_update(FunctionState& state, ViewState& view_state, unsigned int
     }
 
     if (UI::CollapsingHeader("Export")) {
-        if (UI::Button("Render High Precision Popup")) {
-            view_state.wants_high_precision = true;
-        }
         static int export_w = 1920;
         static int export_h = 1080;
 
@@ -462,7 +459,7 @@ void render_and_update(FunctionState& state, ViewState& view_state, unsigned int
                     else {
                         if (ImGui::Button("Save to PNG")) {
                             std::string filename = "high_precision_" + std::to_string(view_state.hp_width) + "x" + std::to_string(view_state.hp_height) + ".png";
-                            export_plot_to_png(view_state.hp_width,view_state.hp_height,"complex-plot-high-precision");
+                            export_plot_to_png(view_state.hp_cpu_buffer.data(),view_state.hp_width,view_state.hp_height,"complex-plot-high-precision.png");
                             view_state.show_export_success = true; 
                         }
                     }

src/interactions/export.cpp

@@ -10,12 +10,9 @@
 #include <emscripten.h>
 #endif
 
-void export_plot_to_png(const int width, const int height, const char* filename) {
-	unsigned char* pixels = new unsigned char[width * height * 4];
-	glReadPixels(0, 0, width, height, GL_RGBA, GL_UNSIGNED_BYTE, pixels);
+void export_plot_to_png(unsigned char* pixels, const int width, const int height, const char* filename) {
 	stbi_set_flip_vertically_on_load(true);
 	stbi_write_png(filename, width, height, 4, pixels, width * 4);
-	delete[] pixels;
 #ifdef __EMSCRIPTEN__
     EM_ASM_({
         const js_filename = UTF8ToString($0);

src/interactions/export.h

@@ -1,2 +1,2 @@
 #pragma once
-void export_plot_to_png(const int width, const int height, const char* filename);
+void export_plot_to_png(unsigned char* pixels, const int width, const int height, const char* filename);

src/main.cpp

@@ -124,6 +124,8 @@ void export_to_png(AppContext* ctx, int target_width, int target_height, const c
 	glGenTextures(1, &texture);
 	glBindTexture(GL_TEXTURE_2D, texture);
 	glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA, target_width, target_height, 0, GL_RGBA, GL_UNSIGNED_BYTE, NULL);
+	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
+    glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
 	glFramebufferTexture2D(GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D, texture, 0);
 
 	glGenRenderbuffers(1, &rbo);
@@ -141,6 +143,7 @@ void export_to_png(AppContext* ctx, int target_width, int target_height, const c
 	if (ctx->function_state->is_3d) glEnable(GL_DEPTH_TEST);
 	else glDisable(GL_DEPTH_TEST);
 
+	draw_scene(ctx, (float)target_width, (float)target_height);
 
 	glPixelStorei(GL_PACK_ALIGNMENT, 1);
 	unsigned char* pixels = new unsigned char[target_width * target_height * 4];