xtensor-stack / xtensor-python

Python bindings for xtensor
BSD 3-Clause "New" or "Revised" License
348 stars 58 forks source link

Doesn't install the python binding #271

Closed yurivict closed 3 years ago

yurivict commented 3 years ago

Version 0.26.0 only installs these files:

include/xtensor-python/pyarray.hpp
include/xtensor-python/pyarray_backstrides.hpp
include/xtensor-python/pycontainer.hpp
include/xtensor-python/pynative_casters.hpp
include/xtensor-python/pystrides_adaptor.hpp
include/xtensor-python/pytensor.hpp
include/xtensor-python/pyvectorize.hpp
include/xtensor-python/xtensor_python_config.hpp
include/xtensor-python/xtensor_type_caster_base.hpp
lib/cmake/xtensor-python/xtensor-pythonConfig.cmake
lib/cmake/xtensor-python/xtensor-pythonConfigVersion.cmake
lib/cmake/xtensor-python/xtensor-pythonTargets.cmake

The actual python binding files are missing.

SylvainCorlay commented 3 years ago

Thanks for opening the issues on various xtensor repositories.

This one seems like the expected behavior. Xtensor-python does not include python bindings. It is a header-only library that allows you to operate on numpy arrays in-place using xtensor syntax, in C++ extensions.

yurivict commented 3 years ago

Perhaps you should consider to correct the project description to how you described it above. The current description says that it is python bindings.

SylvainCorlay commented 3 years ago

Well these are python bindings but not in the way you meant it! It does not include a python package.

yurivict commented 3 years ago

Thank you for your explanation.