New API

.github/changed-files.yml

@@ -2,13 +2,12 @@ golang:
   - goicicle/**/*.go'
   - go.mod
 rust:
-  - src/**/*.rs
-  - build.rs
-  - Cargo.toml
+  - wrappers/rust
 cpp:
   - icicle/**/*.cu
   - icicle/**/*.cuh
   - icicle/**/*.cpp
   - icicle/**/*.hpp
   - icicle/**/*.c
-  - icicle/**/*.h
+  - icicle/**/*.h
+  - icicle/CMakeLists.txt

.github/workflows/main-build.yml

@@ -46,7 +46,9 @@ jobs:
     - name: Checkout Repo
       uses: actions/checkout@v3
     - name: Build Rust
+      working-directory: ./wrappers/rust
       if: needs.check-changed-files.outputs.rust == 'true' || needs.check-changed-files.outputs.cpp_cuda == 'true'
+      # Building from the root workspace will build all members of the workspace by default
       run: cargo build --release --verbose
 
   build-rust-windows:
@@ -65,9 +67,11 @@ jobs:
         # https://docs.nvidia.com/cuda/archive/12.0.0/cuda-installation-guide-microsoft-windows/index.html
         sub-packages: '["cudart", "nvcc", "thrust", "visual_studio_integration"]'
     - name: Build Rust Targets
+      working-directory: ./wrappers/rust
       if: needs.check-changed-files.outputs.rust == 'true' || needs.check-changed-files.outputs.cpp_cuda == 'true'
       env:
         CUDA_PATH: ${{steps.cuda-toolkit.outputs.CUDA_PATH}}
+      # Building from the root workspace will build all members of the workspace by default
       run: cargo build --release --verbose
 
   build-golang-linux:

.github/workflows/main-format.yml

@@ -17,6 +17,7 @@ jobs:
     - name: Checkout
       uses: actions/checkout@v3
     - name: Check rustfmt
+      working-directory: ./wrappers/rust
       run: if [[ $(cargo fmt --check) ]]; then echo "Please run cargo fmt"; exit 1; fi
     # - name: Check clippy
     #   run: cargo clippy --no-deps --all-features --all-targets
@@ -37,5 +38,4 @@ jobs:
     - name: Checkout
       uses: actions/checkout@v3
     - name: Check clang-format
-      run: unformatted_files=$(find ./ -path ./icicle/build -prune -o -path ./target -prune -iname *.h -or -iname *.cuh -or -iname *.cu -or -iname *.c -or -iname *.cpp | xargs clang-format --dry-run -ferror-limit=1 -style=file >&2); if [[ $unformatted_files ]]; then echo $unformatted_files; echo "Please run clang-format"; exit 1; fi
-
+      run: if [[ $(find ./ \( -path ./icicle/build -prune -o -path ./**/target -prune \) -iname *.h -or -iname *.cuh -or -iname *.cu -or -iname *.c -or -iname *.cpp | xargs clang-format --dry-run -ferror-limit=1 -style=file 2>&1) ]]; then echo "Please run clang-format"; exit 1; fi

.github/workflows/main-test.yml

@@ -46,13 +46,19 @@ jobs:
     - name: Checkout Repo
       uses: actions/checkout@v3
     - name: Run Rust Tests
+      working-directory: ./wrappers/rust
       if: needs.check-changed-files.outputs.rust == 'true' || needs.check-changed-files.outputs.cpp_cuda == 'true'
+      # Running tests from the root workspace will run all workspace members' tests by default
+      #TODO: remove test-threads once thread safety is finalized
       run: cargo test --release --verbose -- --test-threads=1
 
   test-cpp-linux:
     name: Test C++ on Linux
     runs-on: [self-hosted, Linux, X64, icicle]
     needs: check-changed-files
+    strategy:
+      matrix:
+        curve: [bn254, bls12_381, bls12_377, bw6_761]
     steps:
     - name: Checkout Repo
       uses: actions/checkout@v3
@@ -61,7 +67,7 @@ jobs:
       if: needs.check-changed-files.outputs.cpp_cuda == 'true'
       run: |
         mkdir -p build
-        cmake -DBUILD_TESTS=ON -S . -B build
+        cmake -DBUILD_TESTS=ON -DCURVE=${{ matrix.curve }} -S . -B build
         cmake --build build
     - name: Run C++ Tests
       working-directory: ./icicle/build

.gitignore

@@ -15,3 +15,4 @@
 **/.DS_Store
 **/Cargo.lock
 **/icicle/build/
+**/wrappers/rust/icicle-cuda-runtime/src/bindings.rs