Merge devel branch in master and release iDynTree 2.0.0 #780
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
All the code that was enabled by the IDYNTREE_USES_KDL option was deprecated in iDynTree 1.0 .
Remove IDYNTREE_USES_KDL option
Furthermore, modernize the involved CMake commands.
Migrate to always use the Eigen3::Eigen imported target
Add CMake option to disable tools compilation
Setting CMAKE_POSITION_INDEPENDENT_CODE to true via an option simplify changing the value CMAKE_POSITION_INDEPENDENT_CODE for an external user, for example if it is cross-compiling iDynTree as static library for a target that does not support position independent code.
Set CMAKE_POSITION_INDEPENDENT_CODE to TRUE as a CMake option
These classes were deprecated in iDynTree 1.0, see #622 for more details.
Remove semantics check classes and related methods
This headers were removed, but they are still used in some downstream projects and were not clearly deprecated in iDynTree 1 . For this reason we add them back, with clear deprecation notices.
Add again deprecated headers
Furthermore: * Cleanup IDYNTREE_COMPILE_SHARED legacy variable and just use the idiomatic BUILD_SHARED_LIBS CMake variable. * Build as shared by default on Windows We use CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS to export the majority of symbols on Windows, and GenerateExportHeader for the global variables. For more info check https://blog.kitware.com/create-dlls-on-windows-without-declspec-using-new-cmake-export-all-feature/ .
Add support to compile iDynTree as a shared library on Windows
* Avoid to use the old GenerateExportHeader included with VTK 6 vtk 6.3 installs a GenerateExportHeader CMake module that shadows the official CMake module if find_package(VTK) is invoked. The issue has been solved in vtk >= 7.1, see https://gitlab.kitware.com/vtk/vtk/-/merge_requests/1593 . To ensure that the build works even if find_package(VTK) is invoked and VTK 6 is found, we manually include exactly the GenerateExportHeader that is shipped with CMake, extracting the CMake install prefix from the CMAKE_COMMAND variable This workaround can be removed once we will not support anymore platforms that ship with vtk 6.3 as main vtk version (Ubuntu 18.04).
This is necessary because we now use the `LibXml2::LibXml2` imported target that is available only for CMake >= 3.12 .
Removed unnecessary warning messages
Fix other matlab visualization related issues after removing deprecated methods
…rfaces (#766) * Cleanup KinDynComputations documentation * Cleanup Span constructor so that it can be used with Eigen Vectors * Cleanup Eigen example to make use of the new Span and MatrixView interfaces * Added some SFINAE checks in Span. * Fixed typo in comment * Avoiding to generate bindings for make_span. Co-authored-by: Stefano <stefano.dafarra@gmail.com>
Cleanup size and indices attributes, return values and arguments
Deprecate getSensorIndex with unsigned int instead of removing it
Automatically publish wheels to PyPI
Prevent CentOS to install libraries into lib64, force instead lib
…_wheels Upload only linux_x86_64 wheels instead of manylinux2014
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.