Simplify negative RNG

Author: sami-hatna66

external/Actor.cpp

@@ -68,14 +68,9 @@ SYCL_EXTERNAL void Actor::setSeed(uint newSeed) { seed = newSeed; }
 SYCL_EXTERNAL void Actor::refreshVariations() {
     // Previous RNG output is used as next seed
     seed = randXorShift(seed);
-    float xDirection = float(seed) > 2151000064 ? -1.0f : 1.0f;
+    float randX = (float(seed) * (2.0f / 4294967296.0f)) - 1.0f;
     seed = randXorShift(seed);
-    float yDirection = float(seed) > 2151000064 ? -1.0f : 1.0f;
-
-    seed = randXorShift(seed);
-    float randX = float(seed) * (1.0f / 4294967296.0f) * xDirection;
-    seed = randXorShift(seed);
-    float randY = float(seed) * (1.0f / 4294967296.0f) * yDirection;
+    float randY = (float(seed) * (2.0f / 4294967296.0f)) - 1.0f;
     this->setVariation({randX, randY});
 }
 

src/main.cpp

@@ -197,76 +197,76 @@ int main(int argc, char *argv[]) {
     auto wallsBuf = sycl::buffer<std::array<vecType, 2>>(walls.data(), walls.size());
     auto pathsBuf = sycl::buffer<Path>(paths.data(), paths.size());
 
-    // int delayCounter = 0;
-    // int updateBBoxCounter = 0;
-    // bool isQuit = false;
-    // bool isPause = false;
-    // SDL_Event event;
-
-    // std::vector<int> executionTimes;
-
-    // while (!isQuit) {
-    //     if (SDL_PollEvent(&event)) {
-    //         if (event.type == SDL_QUIT) {
-    //             isQuit = true;
-    //         } else if (event.type == SDL_KEYDOWN &&
-    //                    event.key.keysym.sym == SDLK_SPACE) {
-    //             isPause = !isPause;
-    //         }
-    //     }
-
-    //     if (!isPause) {
-    //         if (delayCounter >= DELAY) {
-    //             delayCounter = 0;
-    //             auto start = std::chrono::high_resolution_clock::now();
-
-    //             if (updateBBoxCounter <= 0) {
-    //                 updateBBox(myQueue, actorBuf);
-    //                 updateBBoxCounter = 20;
-    //             }
-
-    //             updateVariations(myQueue, actorBuf);
-    //             update(myQueue, actorBuf, wallsBuf, pathsBuf);
-
-    //             auto end = std::chrono::high_resolution_clock::now();
-    //             auto duration =
-    //                 std::chrono::duration_cast<std::chrono::milliseconds>(
-    //                     end - start);
-    //             executionTimes.push_back(duration.count());
-    //             // std::cout << "fps: " << (1000.0f / duration.count()) <<
-    //             // std::endl;
-
-    //             sycl::host_accessor<Actor, 1, sycl::access::mode::read> actorHostAcc(actorBuf);
-
-    //             draw(SCALE, render, actorHostAcc, room);
-    //             updateBBoxCounter--;
-    //         } else {
-    //             delayCounter++;
-    //         }
-    //     }
-    // }
+    int delayCounter = 0;
+    int updateBBoxCounter = 0;
+    bool isQuit = false;
+    bool isPause = false;
+    SDL_Event event;
+
+    std::vector<int> executionTimes;
+
+    while (!isQuit) {
+        if (SDL_PollEvent(&event)) {
+            if (event.type == SDL_QUIT) {
+                isQuit = true;
+            } else if (event.type == SDL_KEYDOWN &&
+                       event.key.keysym.sym == SDLK_SPACE) {
+                isPause = !isPause;
+            }
+        }
 
-    // executionTimes.erase(executionTimes.begin());
-    // float count = static_cast<float>(executionTimes.size());
-    // float mean =
-    //     std::accumulate(executionTimes.begin(), executionTimes.end(), 0.0) /
-    //     count;
-    // std::cout << "Mean execution time: " << mean << std::endl;
-
-    // close(win, render);
-
-    // For Profiling
-    int updateBBoxCounterr = 0;
-    for (int x = 0; x < 500; x++) {
-        if (updateBBoxCounterr <= 0) {
-            updateBBox(myQueue, actorBuf);
-            updateBBoxCounterr = 20;
+        if (!isPause) {
+            if (delayCounter >= DELAY) {
+                delayCounter = 0;
+                auto start = std::chrono::high_resolution_clock::now();
+
+                if (updateBBoxCounter <= 0) {
+                    updateBBox(myQueue, actorBuf);
+                    updateBBoxCounter = 20;
+                }
+
+                updateVariations(myQueue, actorBuf);
+                update(myQueue, actorBuf, wallsBuf, pathsBuf);
+
+                auto end = std::chrono::high_resolution_clock::now();
+                auto duration =
+                    std::chrono::duration_cast<std::chrono::milliseconds>(
+                        end - start);
+                executionTimes.push_back(duration.count());
+                // std::cout << "fps: " << (1000.0f / duration.count()) <<
+                // std::endl;
+
+                sycl::host_accessor<Actor, 1, sycl::access::mode::read> actorHostAcc(actorBuf);
+
+                draw(SCALE, render, actorHostAcc, room);
+                updateBBoxCounter--;
+            } else {
+                delayCounter++;
+            }
         }
-        updateVariations(myQueue, actorBuf);
-        update(myQueue, actorBuf, wallsBuf, pathsBuf);
-        sycl::host_accessor<Actor, 1, sycl::access::mode::read> actorHostAcc(actorBuf);
-        updateBBoxCounterr--;
     }
 
+    executionTimes.erase(executionTimes.begin());
+    float count = static_cast<float>(executionTimes.size());
+    float mean =
+        std::accumulate(executionTimes.begin(), executionTimes.end(), 0.0) /
+        count;
+    std::cout << "Mean execution time: " << mean << std::endl;
+
+    close(win, render);
+
+    // // For Profiling
+    // int updateBBoxCounterr = 0;
+    // for (int x = 0; x < 500; x++) {
+    //     if (updateBBoxCounterr <= 0) {
+    //         updateBBox(myQueue, actorBuf);
+    //         updateBBoxCounterr = 20;
+    //     }
+    //     updateVariations(myQueue, actorBuf);
+    //     update(myQueue, actorBuf, wallsBuf, pathsBuf);
+    //     sycl::host_accessor<Actor, 1, sycl::access::mode::read> actorHostAcc(actorBuf);
+    //     updateBBoxCounterr--;
+    // }
+
     return 0;
 }