Merge branch 'main' into mmg/update-test-workflow

Author: matt-graham

.all-contributorsrc

@@ -93,6 +93,27 @@
       "contributions": [
         "bug"
       ]
+    },
+    {
+      "login": "ElisR",
+      "name": "Elis Roberts",
+      "avatar_url": "https://avatars.githubusercontent.com/u/19764906?v=4",
+      "profile": "https://github.com/ElisR",
+      "contributions": [
+        "bug",
+        "doc"
+      ]
+    },
+    {
+      "login": "ASKabalan",
+      "name": "Wassim KABALAN",
+      "avatar_url": "https://avatars.githubusercontent.com/u/83787080?v=4",
+      "profile": "https://github.com/ASKabalan",
+      "contributions": [
+        "code",
+        "review",
+        "test"
+      ]
     }
   ],
   "contributorsPerLine": 7,

.clang-format

@@ -0,0 +1,11 @@
+---
+Language: Cpp
+BasedOnStyle: Google
+ColumnLimit: 110
+IndentWidth: 4
+AccessModifierOffset: -4
+SortIncludes: false # reordering may break existing code
+ConstructorInitializerIndentWidth: 8
+ContinuationIndentWidth: 8
+...
+# LSST C++ style guide: https://developer.lsst.io/cpp/clang-format.html#general

.coveragerc

@@ -2,4 +2,7 @@
 omit = 
     *test_*
     *__init__*
-    *plotting*
+    *plotting*
+exclude_also =
+    ; Exclude functions requiring CUDA from coverage
+    def \w+_cuda(?:|_\w+)\(

.github/release.yml

@@ -0,0 +1,22 @@
+changelog:
+  exclude:
+    labels:
+      - infrastructure
+    authors:
+      - allcontributors
+  categories:
+    - title: ✨ New features
+      labels:
+        - enhancement
+    - title: 📖 Documentation improvements
+      labels:
+        - documentation
+    - title: 🐛 Bug fixes
+      labels:
+        - bug
+    - title: 📦 Dependency updates
+      labels:
+        - dependencies
+    - title: 🛠 Other changes
+      labels:
+        - "*"

.github/workflows/docs.yml

@@ -26,10 +26,7 @@ jobs:
       run: |
         sudo apt install pandoc
         python -m pip install --upgrade pip
-        pip install jaxlib
-        pip install -r requirements/requirements-core.txt
-        pip install -r requirements/requirements-docs.txt
-        pip install .
+        pip install .[docs]
   
     - name: Build Documentation
       run: |

.github/workflows/linting.yml

@@ -0,0 +1,33 @@
+name: Linting
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  linting:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout source
+        uses: actions/checkout@v4
+
+      - name: Cache pre-commit
+        uses: actions/cache@v4
+        with:
+          path: ~/.cache/pre-commit
+          key: pre-commit-${{ '{{' }} hashFiles('.pre-commit-config.yaml') {{ '}}' }}
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.x"
+          cache: pip
+          cache-dependency-path: pyproject.toml
+
+      - name: Install dependencies
+        run: python -m pip install pre-commit
+
+      - name: Run pre-commit
+        run: pre-commit run --all-files --color always --verbose

.github/workflows/tests.yml

@@ -43,8 +43,7 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install -r requirements/requirements-core.txt -r requirements/requirements-tests.txt
-        pip install .
+        pip install .[tests]
     
     - name: Run tests
       run: |

.gitignore

@@ -20,3 +20,18 @@ resources/
 .matrices/
 s2fft.code-workspace
 .tox
+s2fft/_version.py
+
+#c++/cuda/cmake
+venv
+CMakeLists.txt.user
+CMakeCache.txt
+CMakeFiles
+CMakeScripts
+Testing
+Makefile
+cmake_install.cmake
+install_manifest.txt
+compile_commands.json
+CTestTestfile.cmake
+_deps

.gitmodules

@@ -0,0 +1,6 @@
+repos:
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.6.9
+    hooks:
+      - id: ruff
+      - id: ruff-format