Fixes to compile on MacOS with clang.

Author: LTLA

.github/workflows/run-tests.yaml

@@ -8,12 +8,13 @@ name: Run tests
 
 jobs:
   build:
-    name: Run tests
-    runs-on: ubuntu-latest
+    name: ${{ matrix.os }}, ${{ matrix.parallel }}
+    runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
-        parallel: [true, false]
+        parallel: [ 'serial', 'parallel' ] 
+        os: [ 'ubuntu-latest', 'macos-latest' ]
 
     steps:
     - uses: actions/checkout@v4
@@ -31,7 +32,7 @@ jobs:
         cache: 'pip'
 
     - name: Turn off parallelization flags
-      if: ${{ !matrix.parallel }}
+      if: ${{ matrix.parallel== 'serial' }}
       run: |
         cat tests/lib/CMakeLists.txt | grep -v "TEST_CUSTOM_PARALLEL" > .tmp
         cat .tmp

include/tatami_python/UnknownMatrix.hpp

@@ -83,10 +83,10 @@ class UnknownMatrix : public tatami::Matrix<Value_, Index_> {
         tatami::can_cast_Index_to_container_size<pybind11::array_t<Index_> >(std::max(my_nrow, my_ncol));
 
         auto sparse = my_module.attr("is_sparse")(my_seed);
-        my_sparse = sparse.cast<bool>();
+        my_sparse = sparse.template cast<bool>();
 
         auto grid = my_module.attr("chunk_grid")(my_seed);
-        auto bounds = grid.attr("boundaries").cast<pybind11::tuple>();
+        auto bounds = grid.attr("boundaries").template cast<pybind11::tuple>();
         if (bounds.size() != 2) {
             auto ctype = get_class_name(seed);
             throw std::runtime_error("'chunk_grid(<" + ctype + ">).boundaries' should be a tuple of length 2");

include/tatami_python/sparse_matrix.hpp

@@ -90,7 +90,7 @@ void parse_Sparse2darray(const pybind11::object& matrix, Value_* const vbuffer,
     if (pybind11::isinstance<pybind11::none>(raw_svt)) {
         return;
     }
-    auto svt = raw_svt.cast<pybind11::list>();
+    auto svt = raw_svt.template cast<pybind11::list>();
 
     const auto shape = get_shape<Index_>(matrix);
     const auto NR = shape.first;
@@ -102,18 +102,18 @@ void parse_Sparse2darray(const pybind11::object& matrix, Value_* const vbuffer,
             continue;
         }
 
-        auto inner = raw_inner.cast<pybind11::tuple>();
+        auto inner = raw_inner.template cast<pybind11::tuple>();
         if (inner.size() != 2) {
             auto ctype = get_class_name(matrix);
             throw std::runtime_error("each entry of '<" + ctype + ">.contents' should be a tuple of length 2 or None");
         }
 
-        auto iinput = inner[0].cast<pybind11::array>();
+        auto iinput = inner[0].template cast<pybind11::array>();
         if (ibuffer != NULL) {
             dump_to_buffer(iinput, ibuffer);
         }
         if (vbuffer != NULL) {
-            auto vinput = inner[1].cast<pybind11::array>();
+            auto vinput = inner[1].template cast<pybind11::array>();
             dump_to_buffer(vinput, vbuffer);
         }
 

include/tatami_python/utils.hpp

@@ -26,22 +26,22 @@ inline std::string get_class_name(const pybind11::object& incoming) {
     if (!pybind11::hasattr(cls, "__name__")) {
         return "unnamed";
     }
-    return cls.attr("__name__").cast<std::string>();
+    return cls.attr("__name__").template cast<std::string>();
 }
 
 template<typename Index_>
 std::pair<Index_, Index_> get_shape(const pybind11::object& obj) {
     auto shape = obj.attr("shape");
-    auto tup = shape.cast<pybind11::tuple>();
+    auto tup = shape.template cast<pybind11::tuple>();
     if (tup.size() != 2) {
         auto ctype = get_class_name(obj);
         throw std::runtime_error("'<" + ctype + ">.shape' should return an integer vector");
     }
 
     // We use pybind11's own size type that it uses for NumPy shapes.
     // TODO: check if pybind11::cast() throws an error if it is outside of the range of the target type.
-    auto raw_nrow = tup[0].cast<pybind11::ssize_t>();
-    auto raw_ncol = tup[1].cast<pybind11::ssize_t>();
+    auto raw_nrow = tup[0].template cast<pybind11::ssize_t>();
+    auto raw_ncol = tup[1].template cast<pybind11::ssize_t>();
     if (raw_nrow < 0 || raw_ncol < 0) {
         auto ctype = get_class_name(obj);
         throw std::runtime_error("'dim(<" + ctype + ">)' should contain two non-negative integers");