Merge pull request #359 from dora-rs/minor-refactoring

Remove `Ros2Value` encapsulation of `ArrayData`

.github/workflows/ci.yml

@@ -18,11 +18,32 @@ jobs:
         platform: [ubuntu-latest, macos-latest, windows-latest]
       fail-fast: false
     runs-on: ${{ matrix.platform }}
-    timeout-minutes: 30
+    timeout-minutes: 60
     steps:
       - uses: actions/checkout@v3
       - uses: r7kamura/[email protected]
       - run: cargo --version --verbose
+      - name: Free Disk Space (Ubuntu)
+        uses: jlumbroso/free-disk-space@main
+        if: runner.os == 'Linux' 
+        with:
+          # this might remove tools that are actually needed,
+          # if set to "true" but frees about 6 GB
+          tool-cache: false
+
+          # all of these default to true, but feel free to set to
+          # "false" if necessary for your workflow
+          android: true
+          dotnet: true
+          haskell: true
+          large-packages: false
+          docker-images: true
+          swap-storage: false
+      - name: Free disk Space (Windows)
+        if: runner.os == 'Windows' 
+        run: |
+          docker system prune --all -f
+          Remove-Item "C:\Android" -Force -Recurse
       - uses: Swatinem/rust-cache@v2
         with:
           cache-provider: buildjet
@@ -35,7 +56,13 @@ jobs:
       - name: "Build"
         run: cargo build --all
       - name: "Test"
+        # Remove Windows as there is `pdb` linker issue.
+        # See: https://github.com/dora-rs/dora/pull/359#discussion_r1360268497
+        if: runner.os == 'Linux' || runner.os == 'macOS'
         run: cargo test --all
+      - name: "Test"
+        if: runner.os == 'Windows'
+        run: cargo test --all --lib
 
   # Run examples as separate job because otherwise we will exhaust the disk
   # space of the GitHub action runners.
@@ -51,6 +78,27 @@ jobs:
       - uses: actions/checkout@v3
       - uses: r7kamura/[email protected]
       - run: cargo --version --verbose
+      - name: Free Disk Space (Ubuntu)
+        uses: jlumbroso/free-disk-space@main
+        if: runner.os == 'Linux' 
+        with:
+          # this might remove tools that are actually needed,
+          # if set to "true" but frees about 6 GB
+          tool-cache: false
+
+          # all of these default to true, but feel free to set to
+          # "false" if necessary for your workflow
+          android: true
+          dotnet: true
+          haskell: true
+          large-packages: false
+          docker-images: true
+          swap-storage: false
+      - name: Free disk Space (Windows)
+        if: runner.os == 'Windows' 
+        run: |
+          docker system prune --all -f
+          Remove-Item "C:\Android" -Force -Recurse
       - uses: Swatinem/rust-cache@v2
         with:
           cache-provider: buildjet
@@ -122,7 +170,8 @@ jobs:
         env:
           QT_QPA_PLATFORM: offscreen
         run: |
-          source /opt/ros/humble/setup.bash && ros2 run turtlesim turtlesim_node &
+          # Reset only the turtlesim instance as it is not destroyed at the end of the previous job
+          source /opt/ros/humble/setup.bash && ros2 service call /reset std_srvs/srv/Empty &
           cargo run --example python-ros2-dataflow --features="ros2-examples"
 
   bench:
@@ -155,7 +204,7 @@ jobs:
         platform: [ubuntu-latest, macos-latest, windows-latest]
       fail-fast: false
     runs-on: ${{ matrix.platform }}
-    timeout-minutes: 30
+    timeout-minutes: 60
     steps:
       - uses: actions/checkout@v3
       - uses: r7kamura/[email protected]