Merge pull request #47 from SylvainCorlay/versions

Update to pybind11 2.1.0 and xtensor 0.8

Author: JohanMabille

.appveyor.yml

@@ -24,7 +24,7 @@ install:
   - conda info -a
   - conda install pytest -c conda-forge
   - cd test
-  - conda install xtensor==0.7.3 pytest numpy pybind11==2.0.1 -c conda-forge
+  - conda install xtensor==0.8.0 pytest numpy pybind11==2.1.0 -c conda-forge
   - xcopy /S %APPVEYOR_BUILD_FOLDER%\include %MINICONDA%\include
 
 build_script:

.travis.yml

@@ -35,6 +35,15 @@ matrix:
           packages:
             - clang-3.7
       env: COMPILER=clang CLANG=3.7
+    - os: linux
+      addons:
+        apt:
+          sources:
+            - ubuntu-toolchain-r-test
+            - llvm-toolchain-precise-3.8
+          packages:
+            - clang-3.8
+      env: COMPILER=clang CLANG=3.8
     - os: osx
       osx_image: xcode8
       compiler: clang
@@ -59,7 +68,7 @@ install:
     # Useful for debugging any issues with conda
     - conda info -a
     - cd test
-    - conda install xtensor==0.7.3 pytest numpy pybind11==2.0.1 -c conda-forge
+    - conda install xtensor==0.8.0 pytest numpy pybind11==2.1.0 -c conda-forge
     - cp -r $TRAVIS_BUILD_DIR/include/* $HOME/miniconda/include/
 
 script:

README.md

@@ -30,7 +30,7 @@ conda install -c conda-forge xtensor-python
 
 | `xtensor-python`  | `xtensor`  |  `pybind11` |
 |-------------------|------------|-------------|
-| master            |   0.7.3    | ^2.0.0      |
+| master            |   0.8.0    | ^2.1.0      |
 | 0.7.1             |   0.7.3    | ^2.0.0      |
 | 0.7.0             |   0.7.2    | ^2.0.0      |
 | 0.6.1             |   0.7.1    | ^2.0.0      |

include/xtensor-python/pycontainer.hpp

@@ -12,11 +12,11 @@
 #include <functional>
 #include <numeric>
 #include <cmath>
+
 #include "pybind11/pybind11.h"
 #include "pybind11/common.h"
 #include "pybind11/complex.h"
 
-// Because of layout, otherwise xiterator and xtensor_forward are sufficient
 #include "xtensor/xcontainer.hpp"
 
 namespace xt
@@ -91,34 +91,6 @@ namespace xt
 
     namespace detail
     {
-        // TODO : switch on pybind11::is_fmt_numeric when it is available
-        template <typename T, typename SFINAE = void>
-        struct is_fmt_numeric
-        {
-            static constexpr bool value = false;
-        };
-        
-        constexpr int log2(size_t n, int k = 0)
-        {
-            return (n <= 1) ? k : log2(n >> 1, k + 1);
-        } 
-        
-        template <typename T>
-        struct is_fmt_numeric<T, std::enable_if_t<std::is_arithmetic<T>::value>>
-        {
-            static constexpr bool value = true;
-            static constexpr int index = std::is_same<T, bool>::value ? 0 : 1 + (
-                std::is_integral<T>::value ? log2(sizeof(T)) * 2 + std::is_unsigned<T>::value : 8 + (
-                    std::is_same<T, double>::value ? 1 : std::is_same<T, long double>::value ? 2 : 0));
-        };
-
-        template <class T>
-        struct is_fmt_numeric<std::complex<T>>
-        {
-            static constexpr bool value = true;
-            static constexpr int index = is_fmt_numeric<T>::index + 3;
-        };
-
         template <class T>
         struct numpy_traits
         {
@@ -136,7 +108,7 @@ namespace xt
 
             using value_type = std::remove_const_t<T>;
 
-            static constexpr int type_num = value_list[is_fmt_numeric<value_type>::index];
+            static constexpr int type_num = value_list[pybind11::detail::is_fmt_numeric<value_type>::index];
         };
     }