Work on texture class

+ Set up TextureManager class to handle static texture map

.github/workflows/build-test.yml

@@ -6,40 +6,21 @@ on:
   workflow_dispatch:
 
 jobs:
-  Build-Qtk:
+  Ubuntu:
-    strategy:
+    runs-on: ubuntu-latest
-      fail-fast: false
-      matrix:
-        os: [ubuntu-latest, macos-latest]
-        include:
-          - os: ubuntu-latest
-            CMAKE_PARAMS: -DCMAKE_PREFIX_PATH=/home/runner/work/qtk/Qt/6.3.1/gcc_64/
-          - os: macos-latest
-            CMAKE_PARAMS: -DCMAKE_PREFIX_PATH=/home/runner/work/qtk/Qt/6.3.1/gcc_64/ -DASSIMP_NEW_INTERFACE=on
-
-    runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v3
 
       - name: Install Qt
         uses: jurplel/install-qt-action@v2
         with:
-          version: '6.3.1'
+          version: '6.2.3'
 
-      - name: Install Assimp Linux
+      - name: Install Assimp
-        if: matrix.os == 'ubuntu-latest'
-        shell: bash
         run: |
           sudo apt install libassimp-dev -y
 
-      - name: Install Assimp MacOS
-        if: matrix.os == 'macos-latest'
-        shell: bash
-        run: |
-          brew install assimp
-
       - name: Build Qtk
-        shell: bash
         run: |
           mkdir build && cd build
-          cmake .. ${{ matrix.CMAKE_PARAMS }}  && cmake --build .
+          cmake .. -DCMAKE_PREFIX_PATH=/home/runner/work/qtk/Qt/6.2.3/gcc_64/ && cmake --build .

CMakeLists.txt

@@ -27,25 +27,27 @@ if (NOT Qt6_FOUND)
   message(FATAL_ERROR "Specify path to Qt6 with `cmake -DCMAKE_PREFIX_PATH=/path/to/Qt/6.x.x/gcc_64 -S /path/to/qtk -B /path/to/qtk/build && cmake --build /path/to/qtk/build -j $(nprocs)`")
 endif()
 
+# Add our Qt resources.qrc file to our application
+set(SOURCES app/main.cpp)
+qt6_add_big_resources(SOURCES resources.qrc)
+
+add_executable(qtk ${SOURCES})
+
 ################################################################################
 # External Libraries
 ################################################################################
 
 # https://github.com/assimp/assimp/commit/6ac8279977c3a54118551e549d77329497116f66
 find_package(assimp REQUIRED)
-option(ASSIMP_NEW_INTERFACE "Use assimp::assimp as target instead of assimp" OFF)
+
+
+################################################################################
+# Custom Libraries
+################################################################################
+
+# Mainwidget
 include(GenerateExportHeader)
-
+add_library(main-widget SHARED
-################################################################################
-# Final Application
-################################################################################
-
-# Add our Qt resources.qrc file to our application
-set(SOURCES app/main.cpp)
-qt6_add_big_resources(SOURCES resources.qrc)
-qt_add_executable(qtk ${SOURCES})
-
-set(SOURCES
     src/mainwidget.cpp src/mainwidget.h
     src/mainwindow.cpp src/mainwindow.h src/mainwindow.ui
     src/input.cpp src/input.h
@@ -60,28 +62,21 @@ set(SOURCES
     src/scene.cpp src/scene.h
     src/resourcemanager.cpp src/resourcemanager.h
 )
-qt_add_library(main-widget STATIC ${SOURCES})
 
 target_include_directories(main-widget PUBLIC src/)
-if(ASSIMP_NEW_INTERFACE)
+target_link_libraries(main-widget PRIVATE assimp)
-    target_link_libraries(main-widget PRIVATE assimp::assimp)
+target_link_libraries(main-widget PUBLIC Qt${QT_VERSION_MAJOR}::OpenGLWidgets)
-else()
-    target_link_libraries(main-widget PRIVATE assimp)
-endif()
-target_link_libraries(main-widget PUBLIC Qt6::OpenGLWidgets)
-if(WIN32)
-    find_package(OpenGL REQUIRED)
-    target_link_libraries(main-widget PUBLIC OpenGL::GL)
-endif()
 
-target_link_libraries(qtk PUBLIC main-widget)
+################################################################################
+# Final Application
+################################################################################
 
 # Link qtk executable to main main-widget library
+target_link_libraries(qtk PUBLIC main-widget)
+
 set_target_properties(qtk PROPERTIES
     WIN32_EXECUTABLE TRUE
     MACOSX_BUNDLE TRUE
-    MACOSX_BUNDLE_BUNDLE_VERSION ${PROJECT_VERSION}
-    MACOSX_BUNDLE_SHORT_VERSION_STRING ${PROJECT_VERSION_MAJOR}.${PROJECT_VERSION_MINOR}
 )
 
 generate_export_header(main-widget)

src/model.cpp

@@ -31,8 +31,6 @@ Model * Model::getInstance(const char * name)
 
 void ModelMesh::initMesh(const char * vert, const char * frag)
 {
-  initializeOpenGLFunctions();
-
   // Create VAO, VBO, EBO
   mVAO->create();
   mVBO->create();

src/model.h

@@ -16,7 +16,6 @@
 #include <QOpenGLShaderProgram>
 #include <QOpenGLTexture>
 #include <QOpenGLVertexArrayObject>
-#include <QOpenGLFunctions>
 
 // Assimp
 #include <assimp/Importer.hpp>
@@ -45,7 +44,7 @@ struct ModelTexture {
 
 class Model;
 
-class ModelMesh : protected QOpenGLFunctions {
+class ModelMesh {
 public:
   friend Model;
   typedef std::vector<ModelVertex> Vertices;

src/resourcemanager.cpp

@@ -7,18 +7,7 @@
 ##############################################################################*/
 
 #include "resourcemanager.h"
-#include <algorithm>
-#include <string>
-#include <QtGlobal>
-
-static std::string nixPath(std::string path)
-{
-#ifdef Q_OS_WINDOWS
-    std::replace(path.begin(), path.end(), '\\', '/');
-#endif
-    return path;
-}
 
 std::string RM::resourcesDir =
-    std::string(__FILE__).substr(0, nixPath(__FILE__).find("src/"))
+    std::string(__FILE__).substr(0, std::string(__FILE__).find("src/"))
     + "resources/";

src/skybox.cpp

@@ -69,8 +69,6 @@ void Skybox::draw()
 
 void Skybox::init()
 {
-  initializeOpenGLFunctions();
-
   // Set up shader program
   mProgram.create();
   mProgram.addShaderFromSourceFile(QOpenGLShader::Vertex, ":/skybox.vert");

src/skybox.h

@@ -13,13 +13,12 @@
 #include <QOpenGLShaderProgram>
 #include <QOpenGLTexture>
 #include <QOpenGLVertexArrayObject>
-#include <QOpenGLFunctions>
 
 #include <camera3d.h>
 #include <mesh.h>
 
 
-class Skybox : protected QOpenGLFunctions {
+class Skybox {
 public:
   // Delegate this constructor to use default skybox images
   // + This allows creating a skybox with no arguments ( auto s = new Skybox; )