add a basic workflow (#4)

* added a daily.sh script

* added a daily workflow that runs on the devcloud

* added a config file for pre-commit

* added pre-commit to the workflow

* reformat files

* checks now run on ubuntu-latest

* remove unnecessary pre-commit hooks, reformat

* added the codespell hook and fixed spelling

* added a simple Dockerfile

* added a docker_workflow.yml

* small fixes

* added running pre-commit as a first workflow's step

* remove the unused Dockerfile and devcloud workflow

* remove -DENABLE_SYCL flag from the workflow

Author: haichangsi

.github/workflows/docker_workflow.yml

@@ -0,0 +1,52 @@
+# SPDX-FileCopyrightText: Intel Corporation
+#
+# SPDX-License-Identifier: BSD-3-Clause
+
+name: Docker
+
+on:
+  push:
+  pull_request:
+  workflow_dispatch:
+  # 8am UTC is 12am PST, 1am PDT
+  schedule:
+  - cron: '0 8 * * *'
+
+jobs:
+  run_precommit:
+    name: Run pre-commit
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out
+        uses: actions/checkout@v3
+      - name: Setup Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
+          cache: 'pip'
+      - name: Run pre-commit
+        uses: pre-commit/action@v3.0.0
+  run_examples:
+    name: Build and run examples
+    runs-on: ubuntu-latest
+    container:
+      image: intel/oneapi:latest
+      env:
+        CXX: icpx
+        CC: icx
+      volumes:
+        - ${{ github.workspace }}:/src
+    steps:
+      - name: Check out
+        uses: actions/checkout@v3
+      - name: Setup cmake
+        uses: jwlawson/actions-setup-cmake@v1.13
+      - name: Build examples
+        run: |
+          cmake -B build
+          cmake --build build -j
+      - name: Run examples
+        run: |
+          mpirun -n 2 ./build/src/example1
+          mpirun -n 2 ./build/src/example2
+          mpirun -n 2 ./build/src/example3

.pre-commit-config.yaml

@@ -0,0 +1,37 @@
+# SPDX-FileCopyrightText: Intel Corporation
+#
+# SPDX-License-Identifier: BSD-3-Clause
+
+repos:
+
+- repo: https://github.com/pre-commit/mirrors-clang-format
+  rev: v16.0.6
+  hooks:
+   - id: clang-format
+
+- repo: https://github.com/pre-commit/pre-commit-hooks
+  rev: v4.4.0
+  hooks:
+  - id: trailing-whitespace
+  - id: end-of-file-fixer
+  - id: mixed-line-ending
+  - id: check-yaml
+  - id: check-case-conflict
+  - id: check-added-large-files
+    args: ['--maxkb=800']
+
+
+- repo: https://github.com/pycqa/isort
+  rev: 5.12.0
+  hooks:
+  - id: isort
+
+- repo: https://github.com/cheshirekow/cmake-format-precommit
+  rev: v0.6.13
+  hooks:
+  - id: cmake-format
+
+- repo: https://github.com/codespell-project/codespell
+  rev: v2.2.4
+  hooks:
+  - id: codespell

CMakeLists.txt

@@ -11,8 +11,6 @@ project(
   VERSION 0.1
   DESCRIPTION "Distributed ranges tutorial")
 
-
-
 find_package(MPI REQUIRED)
 
 add_subdirectory(src)
@@ -24,9 +22,9 @@ option(ENABLE_FORMAT "Build with format library" ON)
 include(FetchContent)
 
 FetchContent_Declare(
-    distributed-ranges
-    GIT_REPOSITORY https://github.com/oneapi-src/distributed-ranges.git
-    GIT_TAG c618154a1bceda33e5d61e1536179aeaa11b68f4)
+  distributed-ranges
+  GIT_REPOSITORY https://github.com/oneapi-src/distributed-ranges.git
+  GIT_TAG c618154a1bceda33e5d61e1536179aeaa11b68f4)
 FetchContent_MakeAvailable(distributed-ranges)
 
 FetchContent_Declare(

README.md

@@ -45,7 +45,7 @@ In case your environment is not configured properly or you just prefer a hassle-
  git clone https://github.com/mateuszpn/distributed-ranges-tutorial
  cd distributed-ranges-tutorial
  docker run -it -v $(pwd):/custom-directory-name -u root docker.io/intel/oneapi:latest /bin/bash
- cd custom-directory-name 
+ cd custom-directory-name
  CXX=icpx CC=icx cmake -B build -DENABLE_SYCL=ON
  cmake --build build -j
  ```
@@ -81,12 +81,12 @@ In this way, many of the technical details related to the parallel execution of
 ### Namespaces
 
 General namespace used in the library is `dr::`
-For program using a single node with shared memory available for multiple CPUs and one or more GPUs, data structures and algoritms from `dr::shp::` namespace are provided.
+For program using a single node with shared memory available for multiple CPUs and one or more GPUs, data structures and algorithms from `dr::shp::` namespace are provided.
 For distributed memory model, use the `dr::mhp::` namespace.
 
 ### Data structures
 
-Content of distributes-ranges' data structures is distributed over available nodes. For example, segments of `dr::mhp::distributed_vector` are located in memory of different nodes (mpi processes). Still, global view of the `distributed_vector` is uniform, with contigous indices.
+Content of distributes-ranges' data structures is distributed over available nodes. For example, segments of `dr::mhp::distributed_vector` are located in memory of different nodes (mpi processes). Still, global view of the `distributed_vector` is uniform, with contiguous indices.
 <!-- TODO: some pictures here -->
 
 #### Halo concept
@@ -97,7 +97,7 @@ To support this situation, the concept of halo was introduced. A halo is an area
 
 ### Algorithms
 
-Follwing algorithms are included in distributed-ranges, both in mhp and shp versions:
+Following algorithms are included in distributed-ranges, both in mhp and shp versions:
 
 ```cpp
  copy()
@@ -121,7 +121,7 @@ The examples should be compiled with SYCL compiler and run with.
 mpirun -n N ./build/src/example_name
 ```
 
-where `N` - number of MPI processes. Replace _example_name_ with appropiate name of a file tu run.
+where `N` - number of MPI processes. Replace _example_name_ with appropriate name of a file tu run.
 
 ### Example 1
 
@@ -140,13 +140,13 @@ The example shows the distributed nature of dr data structures. The distributed_
 
 [./src/example3.cpp](src/example3.cpp)
 
- The example simulates the elementary 1-d cellular automaton (ECA). Description of what the automaton is and how it works can be found in [wikipedia](https://en.wikipedia.org/wiki/Elementary_cellular_automaton). Visulisation of the automaton work is available in [ASU team webpage](https://elife-asu.github.io/wss-modules/modules/1-1d-cellular-automata).
+ The example simulates the elementary 1-d cellular automaton (ECA). Description of what the automaton is and how it works can be found in [wikipedia](https://en.wikipedia.org/wiki/Elementary_cellular_automaton). Visualisation of the automaton work is available in [ASU team webpage](https://elife-asu.github.io/wss-modules/modules/1-1d-cellular-automata).
 
  The ECA calculates the new value of a cell using old value of the cell and old values of the cell's neighbors. Therefore a halo of 1-cell width is used, to get access to neighboring cells' values when the loop eaches end of local segment of a vector.
  Additionally, a use of a subrange is presented, and `transform()` function, which puts transformed values of input structure to the output structure, element by element. The transforming function is given as lambda `newvalue`.
  _Please note: after each loop the vector content is printed with `fmt::print()`. The formatter function for `distributed_vector` is rather slow, as it gets the vector element by element, both from local node and remote nodes. You can think about customised, more effective way of results presentation._
 
 <!--
-Consider adding one more example: 
+Consider adding one more example:
 *Simple 2-D operation - Find a pattern in the randomly filled array*
 -->

requirements.txt

@@ -0,0 +1,5 @@
+# SPDX-FileCopyrightText: Intel Corporation
+#
+# SPDX-License-Identifier: BSD-3-Clause
+
+pre-commit==3.4.0

scripts/build_run.sh

@@ -0,0 +1,15 @@
+#! /bin/bash
+
+# SPDX-FileCopyrightText: Intel Corporation
+#
+# SPDX-License-Identifier: BSD-3-Clause
+
+export CXX=icpx
+export CC=icx
+
+cmake -B build
+cmake --build build -j
+
+mpirun -n 2 ./build/src/example1
+mpirun -n 2 ./build/src/example2
+mpirun -n 2 ./build/src/example3

security.md

@@ -1,5 +1,5 @@
 # Security Policy
-Intel is committed to rapidly addressing security vulnerabilities affecting our customers and providing clear guidance on the solution, impact, severity and mitigation. 
+Intel is committed to rapidly addressing security vulnerabilities affecting our customers and providing clear guidance on the solution, impact, severity and mitigation.
 
 ## Reporting a Vulnerability
 Please report any security vulnerabilities in this project utilizing the guidelines [here](https://www.intel.com/content/www/us/en/security-center/vulnerability-handling-guidelines.html).

src/CMakeLists.txt

@@ -9,7 +9,7 @@ if(ENABLE_CUDA)
   add_compile_options(-fsycl-targets=nvptx64-nvidia-cuda
                       -Wno-error=unknown-cuda-version)
   add_link_options(-fsycl-targets=nvptx64-nvidia-cuda
-                    -Wno-error=unknown-cuda-version)
+                   -Wno-error=unknown-cuda-version)
 endif()
 
 add_executable(example1 example1.cpp)

src/example1.cpp

@@ -29,4 +29,4 @@ int main(int argc, char **argv) {
   mhp::finalize();
 
   return 0;
-}
+}

src/example2.cpp

@@ -12,7 +12,7 @@ int main(int argc, char **argv) {
   mhp::init(sycl::default_selector_v);
 
   fmt::print(
-      "Hello, World! Distributed ranges proces is running on rank {} / {} on "
+      "Hello, World! Distributed ranges process is running on rank {} / {} on "
       "host {}\n",
       mhp::rank(), mhp::nprocs(), mhp::hostname());