[WIP] Playtime Support

.github/workflows/cmake.yml

@@ -71,6 +71,30 @@ jobs:
       run: |
         echo "build-output-dir=${{ github.workspace }}/build" >> "$GITHUB_OUTPUT"
 
+    - name: Install Clang and Libraries
+      if: matrix.os == 'ubuntu-latest' && matrix.c_compiler == 'clang'
+      run: |
+        sudo apt-get update
+        sudo apt-get install -y clang libc++-dev libc++abi-dev
+
+    - name: Set Clang 16 as Default
+      if: matrix.os == 'ubuntu-24.04' && matrix.c_compiler == 'clang'
+      run: |
+        sudo update-alternatives --install /usr/bin/clang clang /usr/bin/clang-16 100
+        sudo update-alternatives --install /usr/bin/clang++ clang++ /usr/bin/clang++-16 100
+        sudo update-alternatives --set clang /usr/bin/clang-16
+        sudo update-alternatives --set clang++ /usr/bin/clang++-16
+
+    - name: Check Clang Settings
+      if: matrix.os == 'ubuntu-latest' && matrix.c_compiler == 'clang'
+      run: |
+        clang --version
+        clang++ --version
+        echo "Checking clang headers..."
+        clang++ -v -E -x c++ /dev/null
+        echo "check lld..."
+        ldd --version
+
     - name: Configure CMake
       # Configure CMake in a 'build' subdirectory. `CMAKE_BUILD_TYPE` is only required if you are using a single-configuration generator such as make.
       # See https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html?highlight=cmake_build_type
@@ -83,7 +107,7 @@ jobs:
 
     - name: Build
       # Build your program with the given configuration. Note that --config is needed because the default Windows generator is a multi-config generator (Visual Studio generator).
-      run: cmake --build ${{ steps.strings.outputs.build-output-dir }} --config ${{ matrix.build_type }}
+      run: cmake --build ${{ steps.strings.outputs.build-output-dir }} --config ${{ matrix.build_type }} --verbose
 
     - name: Test
       working-directory: ${{ steps.strings.outputs.build-output-dir }}

CMakeLists.txt

@@ -197,7 +197,8 @@ elseif(APPLE)
 elseif(LINUX)
     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DGUID_STDLIB -std=c++17")
 
-    if(CLANG)
+    if (CMAKE_CXX_COMPILER MATCHES "clang")
+        message(STATUS "Detected Clang compiler: ${CMAKE_CXX_COMPILER}")
         set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -stdlib=libc++")
     endif()
 
@@ -209,7 +210,9 @@ endif()
 
 add_library(GameAnalytics ${LIB_TYPE} ${CPP_SOURCES})
 target_link_libraries(GameAnalytics PRIVATE ${LIBS} PUBLIC ${PUBLIC_LIBS})
-
+message(STATUS "CMAKE_CXX_FLAGS: ${CMAKE_CXX_FLAGS}")
+message(STATUS "CMAKE_EXE_LINKER_FLAGS: ${CMAKE_EXE_LINKER_FLAGS}")
+message(STATUS "CMAKE_SHARED_LINKER_FLAGS: ${CMAKE_SHARED_LINKER_FLAGS}")
 # --------------------------- Google Test Setup --------------------------- #
 
 # Set Project Name

include/GameAnalytics/GameAnalytics.h

@@ -102,19 +102,33 @@ namespace gameanalytics
          static void startSession();
          static void endSession();
 
-         static std::string getRemoteConfigsValueAsString(std::string const& key);
-         static std::string getRemoteConfigsValueAsString(std::string const& key, std::string const& defaultValue);
+        /*
+         template<typename T>
+         static T getRemoteConfigsValue(std::string const& key, T const& defaultValue);
+         */
+
+         static std::string getRemoteConfigsValueAsString(std::string const& key, std::string const& defaultValue = "");
+         static int64_t     getRemoteConfigsValueAsInt(std::string const& key, int64_t defaultValue = 0);
+         static uint64_t    getRemoteConfigsValueAsUInt(std::string const& key, uint64_t defaultValue = 0);
+         static bool        getRemoteConfigsValueAsBool(std::string const& key, bool defaultValue = false);
+         static double      getRemoteConfigsValueAsFloat(std::string const& key, double defaultValue = 0.0);
+         static std::string getRemoteConfigsValueAsJson(std::string const& key);
 
          static bool        isRemoteConfigsReady();
          static void        addRemoteConfigsListener(const std::shared_ptr<IRemoteConfigsListener> &listener);
          static void        removeRemoteConfigsListener(const std::shared_ptr<IRemoteConfigsListener> &listener);
 
          static std::string getRemoteConfigsContentAsString();
-         static std::string getRemoteConfigsContentAsJson();
+
+         static std::string getUserId();
+         static std::string getExternalUserId();
 
          static std::string getABTestingId();
          static std::string getABTestingVariantId();
 
+         static int64_t getElapsedSessionTime();
+         static int64_t getElapsedTimeFromAllSessions();
+
          // game state changes
          // will affect how session is started / ended
          static void onResume();
@@ -130,4 +144,5 @@ namespace gameanalytics
         static bool isSdkReady(bool needsInitialized, bool warn);
         static bool isSdkReady(bool needsInitialized, bool warn, std::string const& message);
     };
+
 } // namespace gameanalytics

sample/Main.cpp

@@ -1,6 +1,7 @@
 #include <string>
 #include <iostream>
 #include <thread>
+#include <chrono>
 
 #include "GameAnalytics/GameAnalytics.h"
 

source/gameanalytics/GADevice.h

@@ -78,6 +78,7 @@ namespace gameanalytics
                 std::string _osVersion;
                 std::string _deviceModel;
                 std::string _deviceManufacturer;
+                std::string _gpu;
 
                 std::string _writablepath;
                 bool        _writablepathStatus{false};

source/gameanalytics/GAEvents.cpp

@@ -116,7 +116,8 @@ namespace gameanalytics
 
             try
             {
-                int64_t sessionLength  = state.calculateSessionLength();
+                // get session length in seconds
+                int64_t sessionLength  = state.calculateSessionLength<std::chrono::seconds>();
 
                 if(sessionLength < 0ll)
                 {

source/gameanalytics/GAHTTPApi.cpp

@@ -56,7 +56,7 @@ namespace gameanalytics
             std::string gameKey = state::GAState::getGameKey();
 
             // Generate URL
-            std::string url = remoteConfigsBaseUrl + "/" + initializeUrlPath + "?game_key=" + gameKey + "&interval_seconds=0&configs_hash=" + configsHash;
+            std::string url = remoteConfigsBaseUrl + "/" + initializeUrlPath + "?game_key=" + gameKey + "&interval_seconds=0&configs_hash=" + configsHash + "&config_vsn_supported=3";
 
             logging::GALogger::d("Sending 'init' URL: %s", url.c_str());
 

source/gameanalytics/GAHealth.cpp

@@ -24,10 +24,9 @@ namespace gameanalytics
     void GAHealth::doFpsReading(float fps)
     {
         int fpsBucket = std::round(fps);
-        if(fpsBucket >= 0 && fpsBucket < MAX_FPS_COUNT)
-        {
-            _fpsReadings[fpsBucket]++;
-        }
+        fpsBucket = std::clamp(fpsBucket, 0, MAX_FPS_VALUE);
+
+        _fpsReadings[fpsBucket]++;
     }
 
     int GAHealth::getMemoryPercent(int64_t memory)
@@ -65,6 +64,7 @@ namespace gameanalytics
         {
             utilities::addIfNotEmpty(out, "cpu_model", _cpuModel);
             utilities::addIfNotEmpty(out, "hardware",  _hardware);
+            utilities::addIfNotEmpty(out, "gpu_model", _gpuModel);
 
             if(_numCores > 0)
             {
@@ -169,7 +169,6 @@ namespace gameanalytics
                     }
                 }
             );
-
         }
     }
 }

source/gameanalytics/GAHealth.h

@@ -29,7 +29,8 @@ namespace gameanalytics
 
         protected:
 
-            static constexpr size_t MAX_FPS_COUNT      = 120 + 1;
+            static constexpr int    MAX_FPS_VALUE      = 120;
+            static constexpr size_t MAX_FPS_COUNT      = MAX_FPS_VALUE + 1;
             static constexpr size_t MAX_MEMORY_COUNT   = 100 + 1;
 
             static constexpr std::chrono::milliseconds MEMORY_TRACK_FREQ {5000};