moveit: 2.5.5-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository moveit to 2.5.5-1:

upstream repository: https://github.com/ros-planning/moveit2.git
release repository: https://github.com/ros2-gbp/moveit2-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 2.5.4-1

chomp_motion_planner
* Switch to clang-format-14 (#1877 <https://github.com/ros-planning/moveit2/issues/1877>) (#1880 <https://github.com/ros-planning/moveit2/issues/1880>)
  * Switch to clang-format-14
  * Fix clang-format-14
  (cherry picked from commit 7fa5eaf1ac21ab8a99c5adae53bd0a2d4abf98f6)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit

No changes

moveit_chomp_optimizer_adapter

No changes

moveit_common
* Add -Wunused-parameter (#1756 <https://github.com/ros-planning/moveit2/issues/1756>) (#1757 <https://github.com/ros-planning/moveit2/issues/1757>)
  (cherry picked from commit be474ec5ba6d0210379d009d518bdd631cc46ad9)
  Co-authored-by: Chris Thrasher<EMAIL_ADDRESS>* Add -Wunused-function (#1754 <https://github.com/ros-planning/moveit2/issues/1754>) (#1755 <https://github.com/ros-planning/moveit2/issues/1755>)
  (cherry picked from commit ed9c3317bc1335b66afb0b2e7478b95ddb5c4b33)
  Co-authored-by: Chris Thrasher<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit_configs_utils
* Do not add Pilz parameters to MoveIt Configs Utils if Pilz is not used (#1583 <https://github.com/ros-planning/moveit2/issues/1583>) (#2174 <https://github.com/ros-planning/moveit2/issues/2174>)
  (cherry picked from commit 1c7fa52edeef08bf8eb1e9cc73c1b0835aaf17e7)
  Co-authored-by: Stephanie Eng<EMAIL_ADDRESS>* Update default planning configs to use AddTimeOptimalParameterization (#2167 <https://github.com/ros-planning/moveit2/issues/2167>) (#2170 <https://github.com/ros-planning/moveit2/issues/2170>)
  (cherry picked from commit 895e9268bd5d9337bebdede07a7f68a99055a1df)
  Co-authored-by: Anthony Baker<EMAIL_ADDRESS>* Add xacro subsititution class and use it for loading urdf & srdf (backport #1805 <https://github.com/ros-planning/moveit2/issues/1805>) (#1937 <https://github.com/ros-planning/moveit2/issues/1937>)
  * Add xacro subsititution class and use it for loading urdf & srdf (#1805 <https://github.com/ros-planning/moveit2/issues/1805>)
  * Add Xacro substitution type
  * Use Xacro substitution for robot description and robot description semantic
  * Install subsititution folder
  * Default to load_xacro if there's no launch substitution specified in the mappings
  (cherry picked from commit 4bc83c3c9e6bfa9efea8c431794a630fbf27dddc)
  # Conflicts:
  #     moveit_configs_utils/moveit_configs_utils/moveit_configs_builder.py
  * Fix merge conflicts
  ---------
  Co-authored-by: Jafar<EMAIL_ADDRESS>  Co-authored-by: Tyler Weaver<EMAIL_ADDRESS>* Add support for multiple MoveItConfigBuilder instaces (#1807 <https://github.com/ros-planning/moveit2/issues/1807>) (#1808 <https://github.com/ros-planning/moveit2/issues/1808>)
  (cherry picked from commit 25d086cee9a7cf1c95a15ea12a27e5b7cbe50a1f)
  Co-authored-by: Marco Magri<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit_core
* Don't use ament_export_targets from package sub folder (backport #1889 <https://github.com/ros-planning/moveit2/issues/1889>) (#1893 <https://github.com/ros-planning/moveit2/issues/1893>)
* Fix comment formatting (#2276 <https://github.com/ros-planning/moveit2/issues/2276>) (#2278 <https://github.com/ros-planning/moveit2/issues/2278>)
  (cherry picked from commit 83892d6a7cb2f84485ebd96d41adb3acd8c44bee)
  Co-authored-by: Stephanie Eng<EMAIL_ADDRESS>* fix for kinematic constraints parsing (#2267 <https://github.com/ros-planning/moveit2/issues/2267>) (#2268 <https://github.com/ros-planning/moveit2/issues/2268>)
  (cherry picked from commit b0f0f680c3f86b8074d208a1e78c92cfa75cf5ca)
  Co-authored-by: Jorge Nicho<EMAIL_ADDRESS>* Added butterworth_filter_coeff parameter (#2129 <https://github.com/ros-planning/moveit2/issues/2129>)
  * Added butterworth_filter_coeff parameter
  * Added formating like in original PR #2091 <https://github.com/ros-planning/moveit2/issues/2091>
  * Update moveit_core/online_signal_smoothing/src/butterworth_filter.cpp
  Co-authored-by: AndyZe<EMAIL_ADDRESS>  * Update moveit_core/online_signal_smoothing/include/moveit/online_signal_smoothing/butterworth_filter.h
  Co-authored-by: AndyZe<EMAIL_ADDRESS>  * Alphabetized dependencies
  * Update moveit_core/package.xml
  Co-authored-by: AndyZe<EMAIL_ADDRESS>  ---------
  Co-authored-by: andrey <>
  Co-authored-by: AndyZe<EMAIL_ADDRESS>* Fix ci-testing build issues (backport #1998 <https://github.com/ros-planning/moveit2/issues/1998>) (#2002 <https://github.com/ros-planning/moveit2/issues/2002>)
* Fix invalid case style for private member in RobotTrajectory
  (cherry picked from commit 31e07d3d6a6c1d59bca5876cc0acc51abb960997)
* Fix unreachable child logger instance
  (cherry picked from commit 1323d05c89a8815450f8f4edf7a1d7b520871d18)
* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Switch to clang-format-14 (#1877 <https://github.com/ros-planning/moveit2/issues/1877>) (#1880 <https://github.com/ros-planning/moveit2/issues/1880>)
  * Switch to clang-format-14
  * Fix clang-format-14
  (cherry picked from commit 7fa5eaf1ac21ab8a99c5adae53bd0a2d4abf98f6)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>* Cleanup msg includes: Use C++ instead of C header (backport #1844 <https://github.com/ros-planning/moveit2/issues/1844>)
  * Cleanup msg includes: Use C++ instead of C header
  * Remove obsolete include: moveit_msgs/srv/execute_known_trajectory.hpp
* Fix moveit_core dependency on tf2_kdl (#1817 <https://github.com/ros-planning/moveit2/issues/1817>) (#1823 <https://github.com/ros-planning/moveit2/issues/1823>)
  This is a proper dependency, and not only a test dependency. It is still
  needed when building moveit_core with -DBUILD_TESTING=OFF.
  (cherry picked from commit 9f7d6df9cac9b55d10f6fee6c29e41ff1d1bf44c)
  Co-authored-by: Scott K Logan<EMAIL_ADDRESS>* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Add -Wunused-function (#1754 <https://github.com/ros-planning/moveit2/issues/1754>) (#1755 <https://github.com/ros-planning/moveit2/issues/1755>)
  (cherry picked from commit ed9c3317bc1335b66afb0b2e7478b95ddb5c4b33)
  Co-authored-by: Chris Thrasher<EMAIL_ADDRESS>* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, Henning Kayser, Robert Haschke, andrey-pr, mergify[bot]

moveit_hybrid_planning
* Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>) (#2277 <https://github.com/ros-planning/moveit2/issues/2277>)
  (cherry picked from commit 5506dd516a91bc145e462b493668ef8623d43521)
  Co-authored-by: Shobuj Paul<EMAIL_ADDRESS>* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Cleanup msg includes: Use C++ instead of C header (backport #1844 <https://github.com/ros-planning/moveit2/issues/1844>)
  * Cleanup msg includes: Use C++ instead of C header
  * Remove obsolete include: moveit_msgs/srv/execute_known_trajectory.hpp
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, Robert Haschke, mergify[bot]

moveit_kinematics
* Fix ikfast package template (#2195 <https://github.com/ros-planning/moveit2/issues/2195>) (#2199 <https://github.com/ros-planning/moveit2/issues/2199>)
  (cherry picked from commit 21036b58e99876928b46e3cc4603a9eb9b85e11d)
  Co-authored-by: Jafar<EMAIL_ADDRESS>* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Contributors: Robert Haschke, mergify[bot]

moveit_planners

No changes

moveit_planners_chomp
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit_planners_ompl
* Fix Constraint Planning Segfault (#2130 <https://github.com/ros-planning/moveit2/issues/2130>) (#2173 <https://github.com/ros-planning/moveit2/issues/2173>)
  * Fix Constraint Planning Segfault
  * Reuse planner data
  * apply clang formatting
  * apply clang formatting round 2
  * add FIXME note and verbose output of planning graph size
  ---------
  Co-authored-by: Sebastian Jahr<EMAIL_ADDRESS>  (cherry picked from commit 92a7951f74baaf26d07356612a2f5dca0bac5065)
  Co-authored-by: Marq Rasmussen<EMAIL_ADDRESS>* Cleanup msg includes: Use C++ instead of C header (backport #1844 <https://github.com/ros-planning/moveit2/issues/1844>)
  * Cleanup msg includes: Use C++ instead of C header
  * Remove obsolete include: moveit_msgs/srv/execute_known_trajectory.hpp
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, Robert Haschke, mergify[bot]

moveit_plugins

No changes

moveit_resources_prbt_ikfast_manipulator_plugin
* Used C++ style casting for int type (backport #1627 <https://github.com/ros-planning/moveit2/issues/1627>) (#1819 <https://github.com/ros-planning/moveit2/issues/1819>)
  (cherry picked from commit 1f32ab0e43f488e9c5bd1957c7677e302c406df0)
  Co-authored-by: Abhijeet Das Gupta<EMAIL_ADDRESS>* Add -Wunused-parameter (#1756 <https://github.com/ros-planning/moveit2/issues/1756>) (#1757 <https://github.com/ros-planning/moveit2/issues/1757>)
  (cherry picked from commit be474ec5ba6d0210379d009d518bdd631cc46ad9)
  Co-authored-by: Chris Thrasher<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit_resources_prbt_moveit_config

No changes

moveit_resources_prbt_pg70_support

No changes

moveit_resources_prbt_support

No changes

moveit_ros

No changes

moveit_ros_benchmarks

No changes

moveit_ros_control_interface
* Fix parameters for ros2_control namespaces (#1833 <https://github.com/ros-planning/moveit2/issues/1833>) (#1897 <https://github.com/ros-planning/moveit2/issues/1897>)
  Co-authored-by: AndyZe<EMAIL_ADDRESS>  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  (cherry picked from commit 5838ce890975e3a058cdc9ab699b27941374c3a2)
  Co-authored-by: Pablo Iñigo Blasco<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit_ros_move_group
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit_ros_occupancy_map_monitor
* Switch to clang-format-14 (#1877 <https://github.com/ros-planning/moveit2/issues/1877>) (#1880 <https://github.com/ros-planning/moveit2/issues/1880>)
  * Switch to clang-format-14
  * Fix clang-format-14
  (cherry picked from commit 7fa5eaf1ac21ab8a99c5adae53bd0a2d4abf98f6)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: mergify[bot]

moveit_ros_perception
* Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>) (#2277 <https://github.com/ros-planning/moveit2/issues/2277>)
  (cherry picked from commit 5506dd516a91bc145e462b493668ef8623d43521)
  Co-authored-by: Shobuj Paul<EMAIL_ADDRESS>* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, mergify[bot]

moveit_ros_planning
* Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>) (#2277 <https://github.com/ros-planning/moveit2/issues/2277>)
  (cherry picked from commit 5506dd516a91bc145e462b493668ef8623d43521)
  Co-authored-by: Shobuj Paul<EMAIL_ADDRESS>* Update default planning configs to use AddTimeOptimalParameterization (#2167 <https://github.com/ros-planning/moveit2/issues/2167>) (#2170 <https://github.com/ros-planning/moveit2/issues/2170>)
  (cherry picked from commit 895e9268bd5d9337bebdede07a7f68a99055a1df)
  Co-authored-by: Anthony Baker<EMAIL_ADDRESS>* Fix timeout in waitForCurrentState (backport #1899 <https://github.com/ros-planning/moveit2/issues/1899>) (#1938 <https://github.com/ros-planning/moveit2/issues/1938>)
  (cherry picked from commit 2c48478ac9b4ccfa6784f0e3a9cbf92ef1ecd363)
  Co-authored-by: Carlo Rizzardo<EMAIL_ADDRESS>  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>* Switch to clang-format-14 (#1877 <https://github.com/ros-planning/moveit2/issues/1877>) (#1880 <https://github.com/ros-planning/moveit2/issues/1880>)
  * Switch to clang-format-14
  * Fix clang-format-14
  (cherry picked from commit 7fa5eaf1ac21ab8a99c5adae53bd0a2d4abf98f6)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>* Cleanup msg includes: Use C++ instead of C header (backport #1844 <https://github.com/ros-planning/moveit2/issues/1844>)
  * Cleanup msg includes: Use C++ instead of C header
  * Remove obsolete include: moveit_msgs/srv/execute_known_trajectory.hpp
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, Robert Haschke, mergify[bot]

moveit_ros_planning_interface
* 400% speed up to move group interface (#1865 <https://github.com/ros-planning/moveit2/issues/1865>) (#1867 <https://github.com/ros-planning/moveit2/issues/1867>)
  (cherry picked from commit 0642ef064df512566ffb171f5a889f4c7886110d)
  Co-authored-by: azalutsky<EMAIL_ADDRESS>* Cleanup msg includes: Use C++ instead of C header (backport #1844 <https://github.com/ros-planning/moveit2/issues/1844>)
  * Cleanup msg includes: Use C++ instead of C header
  * Remove obsolete include: moveit_msgs/srv/execute_known_trajectory.hpp
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, Robert Haschke, mergify[bot]

moveit_ros_robot_interaction
* Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>) (#2277 <https://github.com/ros-planning/moveit2/issues/2277>)
  (cherry picked from commit 5506dd516a91bc145e462b493668ef8623d43521)
  Co-authored-by: Shobuj Paul<EMAIL_ADDRESS>* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, mergify[bot]

moveit_ros_visualization
* Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>) (#2277 <https://github.com/ros-planning/moveit2/issues/2277>)
  (cherry picked from commit 5506dd516a91bc145e462b493668ef8623d43521)
  Co-authored-by: Shobuj Paul<EMAIL_ADDRESS>* Doxygen tag (backport #1955 <https://github.com/ros-planning/moveit2/issues/1955>) (#1958 <https://github.com/ros-planning/moveit2/issues/1958>)
  * Generate Doxygen Tag
  * Install tagfile in output directory
  * Fix problematic override for Doxygen linking
  (cherry picked from commit 752571e9ff027b3137b9720227681ed6b57e42d6)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, mergify[bot]

moveit_ros_warehouse
* Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>) (#2277 <https://github.com/ros-planning/moveit2/issues/2277>)
  (cherry picked from commit 5506dd516a91bc145e462b493668ef8623d43521)
  Co-authored-by: Shobuj Paul<EMAIL_ADDRESS>* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, mergify[bot]

moveit_runtime

No changes

moveit_servo
* Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>) (#2277 <https://github.com/ros-planning/moveit2/issues/2277>)
  (cherry picked from commit 5506dd516a91bc145e462b493668ef8623d43521)
  Co-authored-by: Shobuj Paul<EMAIL_ADDRESS>* [Servo] CI simplification (backport #1556 <https://github.com/ros-planning/moveit2/issues/1556>) (#1980 <https://github.com/ros-planning/moveit2/issues/1980>)
* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Cleanup msg includes: Use C++ instead of C header (backport #1844 <https://github.com/ros-planning/moveit2/issues/1844>)
  * Cleanup msg includes: Use C++ instead of C header
  * Remove obsolete include: moveit_msgs/srv/execute_known_trajectory.hpp
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: AndyZe, Chris Thrasher, Robert Haschke, mergify[bot]

moveit_setup_app_plugins
* add missing dependencies on config utils (backport #1962 <https://github.com/ros-planning/moveit2/issues/1962>) (#2206 <https://github.com/ros-planning/moveit2/issues/2206>)
  when installing ros-humble-moveit-setup-assistant from debs,
  the package cannot currently run due to this missing depend
  (cherry picked from commit cc635471aadfb9446398ece319ae31c6b72bec86)
  Co-authored-by: Michael Ferguson<EMAIL_ADDRESS>* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Contributors: Robert Haschke, mergify[bot]

moveit_setup_assistant
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, mergify[bot]

moveit_setup_controllers
* add missing dependencies on config utils (backport #1962 <https://github.com/ros-planning/moveit2/issues/1962>) (#2206 <https://github.com/ros-planning/moveit2/issues/2206>)
  when installing ros-humble-moveit-setup-assistant from debs,
  the package cannot currently run due to this missing depend
  (cherry picked from commit cc635471aadfb9446398ece319ae31c6b72bec86)
  Co-authored-by: Michael Ferguson<EMAIL_ADDRESS>* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Robert Haschke, mergify[bot]

moveit_setup_core_plugins
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Contributors: Chris Thrasher

moveit_setup_framework
* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, Robert Haschke, mergify[bot]

moveit_setup_srdf_plugins

No changes

moveit_simple_controller_manager
* Use emulated time in action-based controller (#899 <https://github.com/ros-planning/moveit2/issues/899>) (#1743 <https://github.com/ros-planning/moveit2/issues/1743>)
  (cherry picked from commit b6fcac8055f54012dd9e698be6e06f70613a5abf)
  Co-authored-by: Gaël Écorchard<EMAIL_ADDRESS>* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: mergify[bot]

pilz_industrial_motion_planner
* Pilz multi-group incompatibility (#1856 <https://github.com/ros-planning/moveit2/issues/1856>) (#2306 <https://github.com/ros-planning/moveit2/issues/2306>)
  (cherry picked from commit 5bbe21b3574d3e3ef2a2c681b3962f70c3db7e78)
  Co-authored-by: Marco Magri<EMAIL_ADDRESS>  Co-authored-by: Sebastian Jahr<EMAIL_ADDRESS>* Fx class_loader warnings in PILZ unittests (#2296 <https://github.com/ros-planning/moveit2/issues/2296>) (#2303 <https://github.com/ros-planning/moveit2/issues/2303>)
  (cherry picked from commit d4dcea36d9767605fc1921716a67a0c6e9be2a2e)
  Co-authored-by: Yang Lin<EMAIL_ADDRESS>* fix: resolve bugs in MoveGroupSequenceAction class (main branch) (#1797 <https://github.com/ros-planning/moveit2/issues/1797>) (#1809 <https://github.com/ros-planning/moveit2/issues/1809>)
  * fix: resolve bugs in MoveGroupSequenceAction class
  * style: adopt .clang-format
  Co-authored-by: Marco Magri<EMAIL_ADDRESS>  (cherry picked from commit fca8e9bd3f41e6bff5aadbf75c494b5cc3fa25ee)
  Co-authored-by: Marco Magri<EMAIL_ADDRESS>* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Re-enable clang-tidy check performance-unnecessary-value-param (backport #1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Re-enable clang-tidy check performance-unnecessary-value-param (#1703 <https://github.com/ros-planning/moveit2/issues/1703>)
  * Fix clang-tidy issues (#1706 <https://github.com/ros-planning/moveit2/issues/1706>)
  Co-authored-by: Henning Kayser<EMAIL_ADDRESS>  Co-authored-by: Robert Haschke<EMAIL_ADDRESS>* Contributors: Chris Thrasher, mergify[bot]

pilz_industrial_motion_planner_testutils
* Fix clang compiler warnings (backport of #1712 <https://github.com/ros-planning/moveit2/issues/1712>) (#1896 <https://github.com/ros-planning/moveit2/issues/1896>)
  - Fix warning: definition of implicit copy assignment operator is deprecated
  - Fix warning: expression with side effects will be evaluated
  - Fix warning: passing by value
  - Enable -Werror
  - Fix -Wdelete-non-abstract-non-virtual-dtor
  - Fix more clang warnings
  - Modernize gtest: TYPED_TEST_CASE -> TYPED_TEST_SUITE
  - Fix GoogleTestVerification.UninstantiatedTypeParameterizedTestSuite
  - Add default copy/move constructors/assignment operators
  As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator,
  we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
  - Explicitly declare overrides
  - Add default constructors as they are not implicitly declared anymore
  - Declare selected classes as final
  - Add noexcept specifier to constructors
  - Fixup gmock/gtest warnings
* Use <> for non-local headers (#1765 <https://github.com/ros-planning/moveit2/issues/1765>)
  Unless a header lives in the same or a child directory of the file
  including it, it's recommended to use <> for the #include statement.
  For more information, see the C++ Core Guidelines item SF.12
  https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
  (cherry picked from commit 7a1f2a101f9aeb8557e8a31656bbe1a6d53b430e)
* Contributors: Chris Thrasher, Robert Haschke

Holding for Humble sync


fix(Popup): transfer zIndex & fix floated elements

zIndex transfering
Fixes #4083.
In #3947 a wrapping element was added around .popup to avoid Popper.js warnings and position arrows properly. However, that created an issue with zIndex as it was defined inside a nested element:

A popup content

There were multiple proposals in #4083, however I decided to go with syncing zIndex and avoid hardcoded values in JS code:

A popup content

floated elements
Fixes #4092. As was reported in that issue elements that have float CSS properly were breaking layout, this was fixed by adding display: flex to a wrapping element.
popper shorthand
A wrapping element is accessible as a shorthand now, this allows to apply custom props and styling:
  <Popup
    content={
      <>
        A wrapping element in this Popup will have custom id &{' '}
        zIndex.
      </>
    }
    popper={{ id: 'popper-container', style: { zIndex: 2000 } }}
    trigger={<Button>Open a popup</Button>}
  />
)

✅ An example to docs was also added.

Codecov Report

Merging #4094 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4094   +/-   ##
=======================================
  Coverage   99.75%   99.75%
=======================================
  Files         180      180
  Lines        3231     3241   +10
=======================================
+ Hits         3223     3233   +10
  Misses          8        8

Impacted Files
Coverage Δ

src/modules/Popup/Popup.js
98.90% <100.00%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 26cab91...d37efca. Read the comment docs.


feat: transformScale support { w: number; h: number }

transformScale support { w: number; h: number }
and plz upgrade the react-draggable PR and react-resizable PR.

Could you please add tests?


reload_goal() not called when calling SkeletonIK.start(p_one_time=true)

Godot version:
3.2 branch, source build
OS/device including version:
Windows
Issue description:
SkeletonIK's start() function's parameter p_one_time determines whether to internally process the node, or to call the SkeletonIK's _solve_chain() function directly. However, start(true) doesn't work as intended.
When SkeletonIK::start(p_one_time=false) has been called, SkeletonIK waits to receive NOTIFICATION_INTERNAL_PROCESS, upon which it calls reload_goal() followed by _solve_chain().
When SkeletonIK::start(p_one_time=true) is called, it only calls _solve_chain(), but never calls reload_goal to update the target position.
Steps to reproduce:

Make a SkeletonIK on a mesh targeting a moving Spatial, with a script that calls the SkeletonIK's start(true) in _process().
Run project.

Expected behavior:
On each _process(), the IK bones update to reflect the current target position.
Actual behavior:
The bones remain stuck at the first position that the targets were set when the scene started running.
Minimal reproduction project:
(none)
Solution:
Make SkeletonIK::start(p_one_time=true) call reload_goal() before calling _solve_chain().

Also affects the new Skeleton3D class in master branch.


Text-selection breaks normal selection

What is the problem?

When I start second selection in middle of previous then cursor not allowed appears, and selection not start.
What is the current behavior?

I select some elements. This work very well. After that I start new selection. If I start it above or below previous - this still works well. But if I start it in middle of previous selection - new selection will not start.
This was worked well in 1.7
Please provide the steps to reproduce and create a JSFiddle.

ver 1.7

As you see everything working wery well
ver 2.3

If I start new selection from middle of previus, selection will not start
In fiddle with example (https://jsfiddle.net/Simonwep/wez9o7cu/) this issue also occurs.
Code for v1.7
const selection = new Selection({
    class: 'selection',
    frame: document,
    startThreshold: 10,
    disableTouch: false,
    mode: 'touch',
    tapMode: 'native',
    singleClick: true,
    selectables: [
        ".palette-element"
    ],
    startareas: ['html'],
    boundaries: ['html'],
    selectionAreaContainer: 'body',
    scrollSpeedDivider: 10,
    manualScrollSpeed: 750
})

Code for v2.3 (but i comment all events ant this issue still exist)
const selection = new Selection({
    class: 'selection',
    startThreshold: 10,

    singleTap: {
        allow: true,
        intersect: 'touch'
    },

    allowTouch: false,
    overlap: 'invert',
    selectables: [
        ".palette-element"
    ],
    startareas: ['html'],
    boundaries: ['html'],

    scrolling: {
        speedDivider: 10,
        manualSpeed: 750
    }
})

What is the expected behavior?

work fine like version 1.7
Your environment:
Version (SelectionArea.version): 2.3
Browser: Chrome
OS:  Windows 10

Caused by #99 - a simple fix for that is adding user-select: none; to the container. I'll add that to the docs, I don't consider this an issue yet. I'll take a look at it later.


TransitionControlAction

New action that makes a transition of the control

I'm not sure about the policy of this project, but shouldn't be every public method have XMLDocs?


Problems with Hebrew characters

I got this reported by @heiss who uses SkoHub Vocabs for a small group project:

ich möchte Sie hiermit informieren, dass es ein Problem mit Skohub-vocab
gibt, wenn man hebräische Zeichen nutzt. Dann erscheint die Nachricht:
"No definition in language "undefined" provided."
https://heiss.github.io/skohub-docker-vocabs-correspondence/ulb.uni-muenster.de/vocab/skos/bible.he.html#zohar

GitHub repo: https://github.com/Heiss/skohub-docker-vocabs-correspondence

I have also built in on the test server, where the same problem occurs:
https://test.skohub.io/acka47/skohub-docker-vocabs-correspondence/heads/main/ulb.uni-muenster.de/vocab/skos/bible.he.html

Note to myself: JSON Link in the hebrew repo seems not to be working

I forked the repo and rebuild the vocab: https://sroertgen.github.io/skohub-docker-vocabs-correspondence/ulb.uni-muenster.de/vocab/skos/bible.he.html#kabbalah
Everything seems to be there as expected. I suspect the issue arose from using an old docker image, which was not yet updated with the current state of master (now main).
A simple rebuild should fix the issue
JSON Link is also working

ping @Heiss
I will close this now, feel free to reopen if the issue persists

btw a rebuild is triggered with "Actions" > "Build /public and deploy to gh-pages with docker container " and click "Run Workflow"

https://heiss.github.io/skohub-docker-vocabs-correspondence/ulb.uni-muenster.de/vocab/skos/bible.he.html#letters
Sadly, it`s still broken on my side. I used the latest tag in workflow: https://github.com/Heiss/skohub-docker-vocabs-correspondence/blob/main/.github/workflows/main.yml
The workflow checks for changed files, so it does not work with your suggested way to trigger it.

Yep, now it`s fixed for me, too. Think github cache was a thing. ;) I used another browser before to verify, that there was no caching problem on my side. But i forgot, that github have one.
@acka47 yep, there are missing labels on purpose. We wrote about it in the project report.
Thank you all for your help.


Randomly disconnects, suspects session gets invalidated.

The bot randomly times out after a period of 2 songs (5 min) and stops functioning completely.
I suspect it's the session which gets invalidated. The callback gets ran but the action (i.e. skip) does is not actually performed.
Do you know how One would refresh the session without complete restart?

Plug is dead, I presume you are using the staging branch which users are not even supposed to be on. It's quite likely that this, as well, will be gone soon.
Have a nice christmas.


Section Partial Least Squares: num_comp seems to be not tunable in {r chapter-06-pcr} and {r chapter-06-pls}

I don't any issues with this; can you please provide a minimal reprex (reproducible example)? The goal of a reprex is to make it as easy as possible for me to recreate your problem so that I can fix it: please help me help you!
If you've never heard of a reprex before, start by reading "What is a reprex", and follow the advice further down that page.

I don't any issues with this; can you please provide a minimal reprex (reproducible example)? The goal of a reprex is to make it as easy as possible for me to recreate your problem so that I can fix it: please help me help you!
If you've never heard of a reprex before, start by reading "What is a reprex", and follow the advice further down that page.

I'm working on a  downloaded  "tidyAPM" on my desktop.
I faced several issues when running chunks in "chapter-06-Rmd" one at a time.
Let's start from the very first one (chunk "chapter-06-fig-03").
So:

if I first run "_common.R", and then the chunks in "chapter-06-Rmd" one at a time, when running chunk "chapter-06-fig-03" I get this error:

If I don'run "_common.R" I get this one:

Sorry for the screenshot. I hope they may help.
MDG

I'm working on a  downloaded  "tidyAPM" on my desktop.
I faced several issues when running chunks in "chapter-06-Rmd" one at a time.
Let's start from the very first one (chunk "chapter-06-fig-03").
So:

if I first run "_common.R", and then the chunks in "chapter-06-Rmd" one at a time, when running chunk "chapter-06-fig-03" I get this error:

If I don'run "_common.R" I get this one:

Sorry for the screenshot. I hope they may help.
MDG

With updated version of "recipes" it is working now.
MDG

With updated version of "recipes" it is working now.
MDG


PlutoGrid Next Version Work Sharing. 5.2.0

[ ] Documentation
The documentation was on the Wiki of this repository, but it was rarely updated.
Future documents will be updated at the link below.
https://pluto.weblaze.dev/
[ ] Customize keyboard shortcuts
I will update the default keyboard shortcuts to be able to change.
Also, let's check the operation of the keypad. (Before, it was not applied because it was unknown whether the NumLock key on the keypad was activated.)
https://github.com/bosskmk/pluto_grid/pull/490
[ ] AutoComplete
I am going to add AutoComplete to column filter or select popup.
It was planned to be added in the previous version, but it was not added because things to consider when adding this function have not been confirmed.
AutoComplete behavior is very slow if there is a lot of data, especially if there are many columns.
Therefore, it seems that it should be possible to enable fast search by applying indexing to data by selectively using local storage such as sqlite.
[ ] bool suppressAutoSize
https://github.com/bosskmk/pluto_grid/issues/519
[ ] Mobile enter key action
https://github.com/bosskmk/pluto_grid/discussions/566#discussioncomment-3541235
https://github.com/bosskmk/pluto_grid/issues/562
[x] Currency type column
https://github.com/bosskmk/pluto_grid/pull/578
[ ] The horizontal line is disappeared when web render is html
https://github.com/bosskmk/pluto_grid/issues/529
https://github.com/flutter/flutter/issues/111750
Resolved Flutter 3.4.0-28.0.pre.122
[ ] PlutoGridExport - column group
https://github.com/bosskmk/pluto_grid/issues/559
[ ] Infinite scroll
https://github.com/bosskmk/pluto_grid/issues/579#issuecomment-1244657091
[ ] Tooltip for cell
https://github.com/bosskmk/pluto_grid/issues/552
[ ] Row group. (Development is almost complete.)
https://github.com/bosskmk/pluto_grid/issues/551
https://github.com/bosskmk/pluto_grid/issues/502
https://github.com/bosskmk/pluto_grid/issues/474
https://github.com/bosskmk/pluto_grid/issues/237
[ ] Validation cell values
https://github.com/bosskmk/pluto_grid/discussions/583
https://github.com/bosskmk/pluto_grid/issues/587

I am developing a row group function. I'm working on it to be released in the next version.
https://user-images.githubusercontent.com/5284141/190935155-62b095c7-d0b2-4b64-bedc-cae219e17171.mp4

@bosskmk any chances of https://github.com/bosskmk/pluto_grid/pull/578 being released before 5.2?

@Macacoazul01
It will be released together with version 5.2.
The release date is expected to be as short as a few weeks or as long as a month.

Deployed on PlutoGrid 5.1.2. (https://github.com/bosskmk/pluto_grid/pull/578)
https://weblaze.dev/pluto_grid/build/web/#feature/currency-type-column
thank you


Value "configurationOverrides" should be at 'root' of values.yaml...

...as stated in README and referenced on line 106 of deployment.yaml
Without this fix, the reference in deployment.yaml [ {{ range $configName, $configValue := .Values.configurationOverrides }} ] will not work, because in the values.yaml file the configurationOverrides section is under kafka.  As such, if you try to set some value for configurationOverrides, it will not be picked up by the intended section in deployment.yaml!
The original docs have configurationOverrides as a section of its own - as a result I have moved configurationOverrides outside of the kafka section. Indeed, configurationOverrides has nothing to do with the Kafka settings, as they are specific to the schema-registry application.
Also, added a comment in the values.yaml file for a common CORS issues which is detailed as part of the schema-registry-ui documentation. This does not change the functionality of the chart itself, simply highlight a common gotcha which people wanting to run the schema-registry-ui with schema-registry on Kubernetes will inevitably run into.
@benjigoldberg

/assign @benjigoldberg

/ok-to-test

🤦♂ Already there, missed it in values.yaml


Add expiration date to role membership

Add expiration date to role membership
Sets expiration to the one year anniversary of the role creation
date for imported roles, or 6 months after membership grant for
grants that occur on NEXT.
Passes through user and role creation dates from LDAP.
Adds create dates on users and roles.
Resolves issue #861

don't merge, this is failing in a really obscure way; will investigate tomorrow.


Added support for extra parameter for "array of" fields

Regarding Issue: https://github.com/DamienP33/express-mongoose-generator/issues/4
Added the option to use "fieldname:type:extra" syntax, in this case if extra is 'a' (for array) it'll produce an array of the fieldtype on the mongoose model.
Eg. "-f phone:number:a" will yield "phone: [Number]" in the model.
It works for all the types (even the Array type, might consider removing it?) and any future type (e.g. if Mongoose's Mixed type gets support).

Yeah nice !
I'm sorry, I'm busy now.
I look your work ASAP ;)

Good job, simple and efficient :)
I took the opportunity to add the objectId reference.
It lacks more than update the readme, tests and implement the solution for the interactive mode to create a new version (v3.1.0).


Replace tensor with Eigen's unsupported Tensor module

May also require at least Eigen 3.2.5 in CMakeLists.txt.

Eigen's Tensor module is not yet well documented, so delayed to 0.3.


⚠ ReportPortal Server has degraded performance

In 6558b59, ReportPortal Server (https://reporting.its-telekom.eu/) experienced degraded performance:

HTTP code: 200
Response time: 5301 ms

Resolved: ReportPortal Server performance has improved in 1416787 after 5 minutes.


[Featrue](default value) add pi as default value

Proposed changes
Issue Number: refers https://github.com/apache/doris/issues/34228

add pi as default value, such as:
  CREATE TABLE IF NOT EXISTS t1
  (
      k TINYINT,
      v double not null DEFAULT PI
  )
  UNIQUE KEY(k)
  DISTRIBUTED BY HASH(k)
  PROPERTIES("replication_num" = "1");

run buildall


Cannot build bqplot 0.12.x from source

I started see this error when trying to build bqplot from source from your v0.12.x branch. How to fix? Thanks!
Building wheel for bqplot (pyproject.toml): finished with status 'error'
...
      $ jupyter labextension build .
      Building extension in .
      An error occurred.
      ValueError: Extensions require a devDependency on<EMAIL_ADDRESS>you have a dependency on 3.0.9

cc @maartenbreddels
xref https://github.com/spacetelescope/jdaviz/pull/1314#issuecomment-1125179704

Thanks, this should be fixed upstream https://github.com/jupyterlab/jupyterlab/pull/12571

Thank you!


Unable to retrieve destination properties in mail-client

Dear experts,
We are attempting to send mails using the mail-client package. However,Mail destination information required such as host and port cannot be retrieved during execution.
This is because in the source code https://github.com/SAP/cloud-sdk-js/blob/main/packages/mail-client/src/mail-client.ts#L54-L69, it is retrieving the values from "originalProperties", but in reality there is an additional property "destinationConfiguration" encapsulating the values.
To Reproduce

Set up ...
Configure a mail destination in subaccount with mail.host, mail.port and other required information in additional properties. A sample we used is https://help.sap.com/docs/build-process-automation/sap-build-process-automation/configuring-smtp-mail-destination?version=Cloud

Prepare a CAP project and include the mail-client package.
Pass the destination name into the function "sendMail" along with simple mail config.

Execute
Execute CAP project in hybrid testing or deployed mode.

Confirm
Error message will be caught.

See error
Error will be logged as attached in screenshots.

Screenshots
Error log:

Destination Configurations:

Used Versions:

node version via node -v : v18.14.2
npm version via npm -v : 9.5.0
SAP Cloud SDK version you used as dependency:
"@sap-cloud-sdk/connectivity": "^3.10.0",
"@sap-cloud-sdk/mail-client": "^3.10.0",
"@sap/cds": "^7",
"@sap/xssec": "^3",

Impact / Priority
We intend of making use the same package in all applications that will be sending mails but currently have to resort to a custom solution.
Affected development phase: Development
Impact: Impaired
Additional context
A similar issue is raised in the comment of the blog https://blogs.sap.com/2023/02/14/connecting-on-premise-mail-destination-using-sap-cloud-sdk-javascript-version-3-on-sap-cap/
FYI the same destination can be used by SBPA.

Hi @PPG-BrianChan ,
Thanks for raising this. I was able to reproduce this. I'll create a ticket for this in our backlog and update you once we have a fix.

Hey @deekshas8,
we ran into the same issue.
So we would like to know if there is a "roadmap" when this'll be solved?
Greetings

Hey @deekshas8 ,
I am attempting to send mails using the destination mail service. But I am facing the same error as mentioned by others in the thread.
Kindly suggest me if there is a workaround to fix this issue.
Thanks and Regards,
Paran

This is not planned.


Loading sph audio files

Can audio.load load NIST sphere format (SPH) files?
Hi!
So I have tried to load sph files from tedlium dataset with simply:
torchaudio.load("pathtofile")

The load was successful but when saving it to a .wav file to check if the torchaudio read correctly the audio then I realized it was not working.
My question then is if this type of audio files is currently supported by torchaudio and if so, could you provide some link/code that refers how to? I haven't been able to find anything about torchaudio and sph files (but I know sox supports sph and allows to conversion into wav)
In case someone needs sph files here there are some sample ones https://filesamples.com/formats/sph. I got mines from the TEDLIUM dataset.
Thanks

Hi @jiwidi
Thanks for reporting this.
Could you share the following things?

Your environment: the output of python -m torch.utils.collect_env
code snippet you used to save audio

The correctness of the save function in "sox" backend is never tested with "sph" format. We are moving away from "sox" backend but we do not support "sph" format in "sox_io" backend yet. Maybe we can add "sph" support to "sox_io" backend.

Yes @mthrok!
Environment on my macbook:
Collecting environment information...
PyTorch version: 1.7.0.dev20200807
Is debug build: No
CUDA used to build PyTorch: None

OS: Mac OSX 10.15.5
GCC version: (Homebrew GCC 4.9.4_2) 4.9.4
CMake version: version 3.14.0

Python version: 3.7
Is CUDA available: No
CUDA runtime version: No CUDA
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA

Versions of relevant libraries:
[pip3] numpy==1.19.1
[pip3] torch==1.7.0.dev20200807
[conda] blas                      1.0                         mkl
[conda] mkl                       2019.4                      233
[conda] mkl-include               2020.1                      216
[conda] mkl-service               2.3.0            py37hfbe908c_0
[conda] mkl_fft                   1.1.0            py37hc64f4ea_0
[conda] mkl_random                1.1.1            py37h959d312_0
[conda] numpy                     1.19.1           py37h3b9f5b6_0
[conda] numpy-base                1.19.1           py37hcfb5961_0
[conda] pytorch                   1.7.0.dev20200807         py3.7_0    pytorch-nightly
[conda] torch                     1.6.0                    pypi_0    pypi

The code (with the first sample from the link in first message):
import torchaudio
wave,sample = torchaudio.load("sample1.sph")
torchaudio.save("sample1.wav",wave,sample)

I checked if the sample returned from the loading was the same specified as in the sph (44100) and they match.
I try open the wav file and its now a 2hour snoring version

@jiwidi Thanks, and you are working on torchaudio master branch, right?
I added support for sph format in "sox_io" backend.
Can you try my branch mthrok/sph or cherry pick this commit https://github.com/mthrok/audio/commit/f78ebf9556cf237de9e2fb8b3b156dca90769435
and try "sox_io" backend?
import torchaudio
torchaudio.set_audio_backend("sox_io")
wave, sample = torchaudio.load("sample1.sph")
torchaudio.save("sample1.wav", wave, sample)

I believe this works, and if you can confirm I will merge it to master branch.

@mthrok I can confirm it works :+1: Thank you very much for the quick PR!

@jiwidi Thanks for confirming. I merged the commit to master.


improvement: introduce response observables flow

Hi @peppedeka
what do you think about using merge operator for commands response and clear actions?
Something like this:
this.responses$ = merge(this.redisConnectService.response$, this.currentResponseBs.asObservable()).pipe(       scan((a, c) => c != null ? [...a, c] : [], []),     );
in this way we can remove the subscription inside the component .ts file, method destroy and lines from 23 to 25 as well.
Do you think is a good idea?

Good idea,
i remove useless subscription and implementation of ngDestroy.
i'm follow you hint, used static merge operators but i applied  catchError operator for handling external service call (because inject var can be undefined and generate error inside merge that can be break the flow)
I don't remove currentResponse$ getter because is used by async pipe inside the app.component template

Yes absolutely right,
we still have to add an error handler so for now your fix is fine, but wouldn't it be better to handle the error in  RedisConnectService service?
Sorry but i've not found currentResponse$ in app.component template, if it is necessary keep it, otherwise remove.
Thanks so much!

Sounds good!
i moved observables catching error to redis-connect.service
yes :) currentResponse$ is useless i removed it


⚠ Bofrost.at has degraded performance

In bb2c83c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 966 ms

Resolved: Bofrost.at performance has improved in f3beff5 after 6 minutes.


Added Bio/Avatar - Meng Vong

@4MangoTrees Interesting github ID! Why 4?
If I didn't know the meme, I would've been so confused: "First, I am not a cat." =^_^=


please update config description on lakeys.net

Hi
please update configuration description on lakeys.net, for example there is missing use-fixed-rounds, rounds-fixed-number, reveal-answers, additonal-packages, additional-packages-root, debug-log, debug-database-flush etc.

Hi. Regarding use-fixed-rounds and rounds-fixed-number, these two config options are currently only on the live branch--the latest release of TriviaBot does not yet include them. They will be added to the install page once they reach a stable release.
debug-log is strictly for development. additional-packages, additional-packages-root, and debug-database-flush are deprecated functions.
The config descriptions on the install page are intended to be short and simple, rather than definitive, hence why the above options are omitted. Further documentation is planned in a future release, but this is intentional for the time being.

Hi. Regarding use-fixed-rounds and rounds-fixed-number, these two config options are currently only on the live branch--the latest release of TriviaBot does not yet include them. They will be added to the install page once they reach a stable release.
debug-log is strictly for development. additional-packages, additional-packages-root, and debug-database-flush are deprecated functions.
The config descriptions on the install page are intended to be short and simple, rather than definitive, hence why the above options are omitted. Further documentation is planned in a future release, but this is intentional for the time being.

I see, so if I'm using
git -C /home/Dragy/Discord-Trivia-Bot/ pull origin master -v
for updating, that is not the desired way?

I see, so if I'm using
git -C /home/Dragy/Discord-Trivia-Bot/ pull origin master -v
for updating, that is not the desired way?


Errors are not reported if something else goes wrong

It took me a long while to figure out why my program was not reporting JSON Schema errors! Here's a much-simplified version of what was happening:
ringo:autogen$ cat t.js
var $RefParser = require('json-schema-ref-parser');

$RefParser.dereference('some nonexistent file', function(err, schema) {
  if (err) {
    console.error(err);
  } else {
    console.log(schema);
  }
});

console.log('All done');
process.exit(0);
ringo:autogen$ node t.js
All done
ringo:autogen$

If I comment out that last line (process.exit(0)), then the error from the missing JSON Schema is correctly reported: Error opening file "/Users/mike/git/folio/other/mod-graphql/src/autogen/some nonexistent file" ENOENT: no such file or directory.
I think what's going on here is that, since the dereference callback function is run asynchronously, it doesn't reach the point of reporting the ENOENT error before the program exits. But it does report the errors if the exit is explicit (i.e. fall off the end of the program) rather than explicit.
I think that errors should always be reported when they occur, irrespective of how the program terminates. Or, if that's not possible, there should be a way to await the completion of the dereference callback function before continuing with the rest of the program.

BTW., it behaves the same way when using the promiseish version of the API.

The error that you see in the console when you comment-out process.exit(0) is coming from the line where you log the error to the console (console.error(err)).
See reproduction here: https://runkit.com/bigstickcarpet/json-schema-ref-parser-issue-85

Sorry, evidently I didn't explain this as clearly as I intended! I know, and knew, where the error-message is coming from. The problem is that I do not see the error message if the program exits.

That's just normal Node.js async behavior.   The process.exit() method is synchronous and exits the program immediately without allowing asynchronous operations any time to complete.

Got it. I need the remainder of my program not to follow the dereference invocation, but to be in the callback's no-error branch.
Thanks for helping me think more clearly about this. I agree that the issue should be closed.


Broadcast shape incompatible while training fpn

Below is the full traceback of the error I get. I am training fpn_dcn on my custom data in the pascal voc format. PLEASE help me with this error. Thank you in advance

$ python experiments/fpn/fpn_end2end_train_test.py --cfg experiments/fpn/cfgs/resnet_v1_101_voc_visdrone_trainval_fpn_dcn_end2end_ohem.yaml
('Called with argument:', Namespace(cfg='experiments/fpn/cfgs/resnet_v1_101_voc_visdrone_trainval_fpn_dcn_end2end_ohem.yaml', frequent=100))
{'CLASS_AGNOSTIC': False,
'MXNET_VERSION': 'mxnet',
'SCALES': [(800, 1280)],
'TEST': {'BATCH_IMAGES': 1,
'CXX_PROPOSAL': False,
'HAS_RPN': True,
'NMS': 0.3,
'PROPOSAL_MIN_SIZE': 0,
'PROPOSAL_NMS_THRESH': 0.7,
'PROPOSAL_POST_NMS_TOP_N': 2000,
'PROPOSAL_PRE_NMS_TOP_N': 20000,
'RPN_MIN_SIZE': 0,
'RPN_NMS_THRESH': 0.7,
'RPN_POST_NMS_TOP_N': 2000,
'RPN_PRE_NMS_TOP_N': 12000,
'SOFTNMS_THRESH': 0.6,
'USE_SOFTNMS': True,
'max_per_image': 100,
'test_epoch': 7},
'TEST_SCALES': [[480, 800],
[576, 900],
[688, 1100],
[800, 1200],
[1200, 1600],
[1400, 2000]],
'TRAIN': {'ALTERNATE': {'RCNN_BATCH_IMAGES': 0,
'RPN_BATCH_IMAGES': 0,
'rfcn1_epoch': 0,
'rfcn1_lr': 0,
'rfcn1_lr_step': '',
'rfcn2_epoch': 0,
'rfcn2_lr': 0,
'rfcn2_lr_step': '',
'rpn1_epoch': 0,
'rpn1_lr': 0,
'rpn1_lr_step': '',
'rpn2_epoch': 0,
'rpn2_lr': 0,
'rpn2_lr_step': '',
'rpn3_epoch': 0,
'rpn3_lr': 0,
'rpn3_lr_step': ''},
'ASPECT_GROUPING': True,
'BATCH_IMAGES': 1,
'BATCH_ROIS': -1,
'BATCH_ROIS_OHEM': 512,
'BBOX_MEANS': [0.0, 0.0, 0.0, 0.0],
'BBOX_NORMALIZATION_PRECOMPUTED': True,
'BBOX_REGRESSION_THRESH': 0.5,
'BBOX_STDS': [0.1, 0.1, 0.2, 0.2],
'BBOX_WEIGHTS': array([1., 1., 1., 1.]),
'BG_THRESH_HI': 0.5,
'BG_THRESH_LO': 0.0,
'CXX_PROPOSAL': False,
'ENABLE_OHEM': True,
'END2END': True,
'FG_FRACTION': 0.25,
'FG_THRESH': 0.5,
'FLIP': True,
'RESUME': False,
'RPN_BATCH_SIZE': 256,
'RPN_BBOX_WEIGHTS': [1.0, 1.0, 1.0, 1.0],
'RPN_CLOBBER_POSITIVES': False,
'RPN_FG_FRACTION': 0.5,
'RPN_MIN_SIZE': 0,
'RPN_NEGATIVE_OVERLAP': 0.3,
'RPN_NMS_THRESH': 0.7,
'RPN_POSITIVE_OVERLAP': 0.7,
'RPN_POSITIVE_WEIGHT': -1.0,
'RPN_POST_NMS_TOP_N': 2000,
'RPN_PRE_NMS_TOP_N': 12000,
'SHUFFLE': True,
'begin_epoch': 0,
'end_epoch': 7,
'lr': 0.01,
'lr_factor': 0.1,
'lr_step': '4,6',
'model_prefix': 'fpn_coco',
'momentum': 0.9,
'warmup': True,
'warmup_lr': 0.001,
'warmup_step': 250,
'wd': 0.0001},
'dataset': {'NUM_CLASSES': 13,
'dataset': 'PascalVOC',
'dataset_path': '/scratch/user/k21993/visdrone/VisDrone2018/Code/data/VOCdevkit2007',
'image_set': '2007_trainval',
'proposal': 'rpn',
'root_path': './data',
'test_image_set': '2007_test'},
'default': {'frequent': 100, 'kvstore': 'device'},
'gpus': '0,1',
'network': {'ANCHOR_RATIOS': [0.5, 1, 2],
'ANCHOR_SCALES': [8],
'FIXED_PARAMS': ['conv1',
'bn_conv1',
'res2',
'bn2',
'gamma',
'beta'],
'FIXED_PARAMS_SHARED': ['conv1',
'bn_conv1',
'res2',
'bn2',
'res3',
'bn3',
'res4',
'bn4',
'gamma',
'beta'],
'IMAGE_STRIDE': 32,
'NUM_ANCHORS': 3,
'PIXEL_MEANS': array([103.06, 115.9 , 123.15]),
'RCNN_FEAT_STRIDE': 16,
'RPN_FEAT_STRIDE': [4, 8, 16, 32, 64],
'pretrained': './model/pretrained_model/resnet_v1_101',
'pretrained_epoch': 0},
'output_path': './output/fpn/coco',
'symbol': 'resnet_v1_101_fpn_dcn_rcnn'}
num_images 7009
voc_2007_trainval gt roidb loaded from ./data/cache/voc_2007_trainval_gt_roidb.pkl
append flipped images to roidb
filtered 22 roidb entries: 14018 -> 13996
providing maximum shape [('data', (1, 3, 800, 1280)), ('gt_boxes', (1, 100, 5))] [('label', (1, 255780)), ('bbox_target', (1, 12, 85260)), ('bbox_weight', (1, 12, 85260))]
{'bbox_target': (1L, 12L, 72471L),
'bbox_weight': (1L, 12L, 72471L),
'data': (1L, 3L, 800L, 1088L),
'gt_boxes': (1L, 33L, 5L),
'im_info': (1L, 3L),
'label': (1L, 217413L)}
('lr', 0.01, 'lr_epoch_diff', [4.0, 6.0], 'lr_iters', [27992, 41988])
experiments/fpn/../../fpn/../lib/bbox/bbox_transform.py:82: RuntimeWarning: invalid value encountered in log
targets_dw = np.log(gt_widths / ex_widths)
experiments/fpn/../../fpn/operator_py/fpn_roi_pooling.py:30: RuntimeWarning: invalid value encountered in sqrt
feat_id = np.clip(np.floor(2 + np.log2(np.sqrt(w * h) / 224)), 0, len(self.feat_strides) - 1)
Error in CustomOp.forward: Traceback (most recent call last):
File "/scratch/user/k21993/visdrone/Deformable-ConvNets/mxnet_python27/lib/python2.7/site-packages/mxnet/operator.py", line 987, in forward_entry
aux=tensors[4])
File "experiments/fpn/../../fpn/operator_py/fpn_roi_pooling.py", line 88, in forward
self.assign(out_data[0], req[0], roi_pool)
File "/scratch/user/k21993/visdrone/Deformable-ConvNets/mxnet_python27/lib/python2.7/site-packages/mxnet/operator.py", line 468, in assign
dst[:] = src
File "/scratch/user/k21993/visdrone/Deformable-ConvNets/mxnet_python27/lib/python2.7/site-packages/mxnet/ndarray/ndarray.py", line 444, in setitem
self._set_nd_basic_indexing(key, value)
File "/scratch/user/k21993/visdrone/Deformable-ConvNets/mxnet_python27/lib/python2.7/site-packages/mxnet/ndarray/ndarray.py", line 699, in _set_nd_basic_indexing
value = value.broadcast_to(shape)
File "/scratch/user/k21993/visdrone/Deformable-ConvNets/mxnet_python27/lib/python2.7/site-packages/mxnet/ndarray/ndarray.py", line 1677, in broadcast_to
raise ValueError(err_str)
ValueError: operands could not be broadcast together with remapped shapes[original->remapped]: (2070L, 256L, 7L, 7L) and requested shape (2072L, 256L, 7L, 7L)

Your image must be divided by 32.

Do you mean for each pixel value we do:
pixel_value = pixel_value/32 ?
By the way, this error only seems to be for FPN, I am able to run the code for RFCN. Please let me know why this is happening. Thank you very much for your help :)

No, the image width and height must be divided by 32.

To make sure I understand you correctly, are you saying I downsample each image (width and height) by a factor of 32 before I feed it into training? Why is this so?

The image width and height must be exactly dividable by 32. For example 1024 is OK but 1023 cannot be training because "1024 mod 32 = 0".

Hi, so should I make this change in the config file? If so the given scales are already divisible by 32
'---
MXNET_VERSION: "mxnet"
output_path: "./output/fpn/coco"
symbol: resnet_v1_101_fpn_dcn_rcnn
gpus: '0,1'
CLASS_AGNOSTIC: false
SCALES:\

800\
1280\

TEST_SCALES: [[800, 1280]] # single scale testing\
TEST_SCALES: [[480, 800], [576, 900], [688, 1100], [800, 1200], [1200, 1600], [1400, 2000]] # multi-scale testing`

I chang all my images to 1280*800 and get training successful.

How did you resize smaller images (I have 1000600, 800600 etc) to 1280*800? Did you zero pad the smaller images?

Big resized, small padded.

Thank you very much for your help. I will make the changes and get back to you

I tried your suggestion, I get the same shape error at a different place
Error in CustomOp.forward: Traceback (most recent call last):
File "/home/k21993/visdrone_mxnet/local/lib/python2.7/site-packages/mxnet/operator.py", line 782, in forward_entry
aux=tensors[4])
File "experiments/fpn/../../fpn/operator_py/fpn_roi_pooling.py", line 88, in forward
self.assign(out_data[0], req[0], roi_pool)
File "/home/k21993/visdrone_mxnet/local/lib/python2.7/site-packages/mxnet/operator.py", line 455, in assign
dst[:] = src
File "/home/k21993/visdrone_mxnet/local/lib/python2.7/site-packages/mxnet/ndarray/ndarray.py", line 405, in setitem
value.copyto(self)
File "/home/k21993/visdrone_mxnet/local/lib/python2.7/site-packages/mxnet/ndarray/ndarray.py", line 1635, in copyto
return _internal._copyto(self, out=other)
File "", line 25, in _copyto
File "/home/k21993/visdrone_mxnet/local/lib/python2.7/site-packages/mxnet/_ctypes/ndarray.py", line 92, in _imperative_invoke
ctypes.byref(out_stypes)))
File "/home/k21993/visdrone_mxnet/local/lib/python2.7/site-packages/mxnet/base.py", line 146, in check_call
raise MXNetError(py_str(_LIB.MXGetLastError()))
MXNetError: [06:48:52] src/operator/nn/./../tensor/../elemwise_op_common.h:122: Check failed: assign(&dattr, (*vec)[i]) Incompatible attr in node  at 0-th output: expected (2070,256,7,7), got (2072,256,7,7)
Can you help me with this?

Use a smaller dataset to ensure it is not the dataset's problem.
Use the newest Mxnet, install with 'pip install --pre mxnet-cu80 or 90 or 91.

I tried on fpn deformable conv net on the PASCAL VOC dataset and it still gives errors.

I tried fpn with rcnn only.

yes, fpn rcnn seems to work. I think it might be a bug in their code

Hi @chinakook , I was finally able to get training to run on the images resized so that both height and width are divisible by 32. However, the same error has come up when I run fpn_test.py, even though the test images seem to be of shape 1024x1024 (both divisible by 32). Please help me regarding this

Refer to main repo https://github.com/chinakook/Deformable-ConvNets/tree/dev/fpn
It’s compitable with mx-rcnn and has deploy and demo script

Thank you, I'll give it a try!


Demo link broken on README

After redirecting it 404s.
https://preview.c9users.io/carylandholt/ng-classify-browserify/index.html

Should be up and running now, thanks


Add a link between a person and one or several organizations

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
We want to be able to display the list of teachers member of an organization. For this, we are missing a link between a person and an organization.
Describe the solution you'd like
Add a placeholder on the organization page allowing to add person plugins.
The solution is the same as what is already done to add subjects or organizations to a course.
Discovery, Documentation, Adoption, Migration Strategy
We should also display the organizations to which a teacher is linked on the person page.

This feature request was handled in https://github.com/openfun/richie/pull/670.
We decided to link organizations on a person detail page and not the other way around because we think it will be easier to maintain:

the organization page is created once and then persons are repeatedly added,
cardinality asymetry: few organizations per user but many users per organization.


Aws es signing

Adds AWS ElasticSearch request signing, closes #346,  also merged #333 into my branch as I wanted Go modules and it was already created as a PR.

I've also just noticed that #350 was created yesterday. If required I can use that Go modules instead.

any plans for this? would be great to have ability to use exporter with AWS elasticseach

@JackFazackerley I have taken these changes for a spin on EKS, but haven't had any luck actually reaching our AWS elasticsearch instance. Just wondering if you can confirm that this can support IRSA?
Example: https://aws.amazon.com/blogs/opensource/introducing-fine-grained-iam-roles-service-accounts/
Any examples of how to achieve this in a deployment manifest or the like would be extremely helpful!

Sorry for only just getting back to you on this guys, only just noticed it.
@k1rk if you wanted to use it, my fork should work via the following commands
AWS_ACCESS_KEY_ID="ACCESS_KEY"
export AWS_SECRET_ACCESS_KEY="SECRET_ACCESS_KEY"
go run -mod=vendor main.go tls.go logger.go --es.uri="ES_URI" --log.level="debug" --es.aws --es.aws-region="YOUR_REGION"

Assuming that works you could build and use the binary.
@KyleMasterson From reading the docs it should be possible already if using my fork. As the docs say:

For this feature to work correctly, you’ll need to use an SDK version greater than or equal to the values listed below:

The Go SDK being used is v1.31.3 and the specified version on the docs is v1.23.13 so it should be possible. Note i'm not using IRSA, my credentials are imported via Vault.
If you still wanted an example of what my manifest looks like: (I've added environment vars in the manifest below to work with keys)
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: elasticsearch-exporter
  name: elasticsearch-exporter
  namespace: default
spec:
  progressDeadlineSeconds: 600
  replicas: 1
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      app: elasticsearch-exporter
  strategy:
    rollingUpdate:
      maxSurge: 1
      maxUnavailable: 0
    type: RollingUpdate
  template:
    metadata:
      labels:
        app: elasticsearch-exporter
    spec:
      containers:
        - command:
            - elasticsearch_exporter
            - --es.uri=YOUR_ES_URI
            - --es.all
            - --es.indices
            - --es.indices_settings
            - --es.shards
            - --es.snapshots
            - --es.timeout=30s
            - --web.listen-address=:9108
            - --web.telemetry-path=/metrics
          env:
            - name: AWS_ACCESS_KEY_ID
              value: YOUR_ACCESS_KEY_ID
            - name: AWS_SECRET_ACCESS_KEY
              value: YOUR_SECRET_ACCESS_KEY
            - name: ES_AWS
              value: "true"
            - name: ES_AWS_REGION
              value: YOUR_REGION
          image: elasticsearch-exporter:aws-es-signing
          imagePullPolicy: IfNotPresent
          lifecycle:
            preStop:
              exec:
                command:
                  - /bin/bash
                  - -c
                  - sleep 20
          livenessProbe:
            failureThreshold: 3
            httpGet:
              path: /healthz
              port: http
              scheme: HTTP
            initialDelaySeconds: 5
            periodSeconds: 5
            successThreshold: 1
            timeoutSeconds: 5
          name: elasticsearch-exporter
          ports:
            - containerPort: 9108
              name: http
              protocol: TCP
          readinessProbe:
            failureThreshold: 3
            httpGet:
              path: /healthz
              port: http
              scheme: HTTP
            initialDelaySeconds: 1
            periodSeconds: 5
            successThreshold: 1
            timeoutSeconds: 5
          resources: {}
          securityContext:
            capabilities:
              drop:
                - SETPCAP
                - MKNOD
                - AUDIT_WRITE
                - CHOWN
                - NET_RAW
                - DAC_OVERRIDE
                - FOWNER
                - FSETID
                - KILL
                - SETGID
                - SETUID
                - NET_BIND_SERVICE
                - SYS_CHROOT
                - SETFCAP
            readOnlyRootFilesystem: true
          terminationMessagePath: /dev/termination-log
          terminationMessagePolicy: File
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext:
        runAsNonRoot: true
        runAsUser: 1000
      serviceAccount: default
      serviceAccountName: default
      terminationGracePeriodSeconds: 30```

Thank you for this! I was afraid I'd have to implement this myself 😄 I will test this with IRSA and report back. At least the credential chain seems to prioritise env-credentials, so it should work

I could use these changes. Is there anything I can do to help move this along?

This PR will need to be rebased. Go module support has been merged already.

@sysadmind I rebased and cleaned this up in a separate PR: https://github.com/prometheus-community/elasticsearch_exporter/pull/443


Authentication screen: remove Tchap icon, add ActionBar title

Ok, the scope of the issue #187 is done:
Login is ok.
Registration first step is ok.
But there is still work to do for the complete login/registration flow

The design is correctly introduced here, but we need to display correctly the different steps of the registration flow display (pending email validation, registration disabled...) before merging on develop

I have added the "Wait for email" screen
"Disabled registration" do not lead to login screen anymore


Make "type" prop propagation explicit

Environment

react-advanaced-form: 1.5.11

What
Current behavior
Whenever a new field is created, its type property is defaulted to "text" in both field state and field element. This makes field components like Textarea, Select and Radio abide by the type="text" validation rules, which is unexpected.
Expected behavior
Only Input component must have type="text" by default.
Why
Doing this allows to distinguish between field state's type and the type assigned to a field component. That means developers can assign their custom types to the field components to extend their validation logic, without spamming DOM elements within unsupported type attribute values.
How

Make propagation of the type prop to a field element explicit. Do not inherit from the state.
Assign necessary type prop on a field element explicitly using `enforceProps.

This also relates to the components as Select and Radio. At this point, I think type="text" should be set for Input components only.

Implemented. Will be released under 1.6.


settings: Fix text-overflow inside organization settings > stream announcement & user announcement

Fixed text-overflow of organization settings > New stream announcements  & New user announcements
The overflow will display as ellipsis: ...

Fixes: #24539
CZO: Thread
Screenshots and screen captures:

Before:

New Changes:
Without overflow:

With overflow:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Hey @alya , I have made the changes as discussed on the CZO. Could you please review the PR? Thanks.

Closing this PR since I have merged it with another PR of mine PR #24487, which resolves the bot management modal issue and includes the changes from this PR.


don't make gene names uppercase

Fixes #450
Fixes #459

Codecov Report

Merging #495 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #495      +/-   ##
==========================================
+ Coverage   88.61%   88.64%   +0.02%
==========================================
  Files          34       34
  Lines        4226     4226
==========================================
+ Hits         3745     3746       +1
+ Misses        481      480       -1

Impacted Files
Coverage Δ

scvi/dataset/dataset.py
91.47% <ø> (ø)
:arrow_up:

scvi/inference/autotune.py
75.78% <0%> (+0.21%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f0aca3...d7184d4. Read the comment docs.

Hi Adam, why don't you want to make gene uppercase? We need some sort of standardization because for spatial-sequencing experiments (GimVI for instance) then we match the genes between two datasets by their names. If the gene names are not all uppercase then the matching will fail.

Hi Adam, why do you want to make gene uppercase? We need some sort of standardization because for spatial-sequencing experiments (GimVI for instance) then we match the genes between two datasets by their names. If the gene names are not all uppercase then the matching will fail.

I think the idea is that there are standards to gene names that people expect, and making them uppercase breaks that. I think we need to do a bit more research before implementing these changes.

@ANazaret how do you feel if the user uppercases the gene names themselves in each dataset object before merging?

@ANazaret how do you feel if the user uppercases the gene names themselves in each dataset object before merging?

I think they won't do it and it can fail. Probably the better thing is to change directly the function matching the names so that this function is not sensitive to the case.
That is replacing g by g.lower() for instance in this snippet.
https://github.com/YosefLab/scVI/blob/05920d1f85daa362d4fb694e588ab090bc84e207/scvi/dataset/cortex.py#L73-L75

Is this PR going to be merged? Upper-casing gene names is an issue when working with non-human datasets.

@ANazaret I think we will add an option in populate_from_data that uppercases the gene names with the default being no uppercase.
There will be some instances where this is not desirable, but downstream analysis (DE for example) is much more difficult when gene names are changed. We could do what you suggested but then how would we set the gene_names attribute of a concatenated dataset?

@adamgayoso
I feel like changing feature names is something that should be done outside of a machine learning model, so it makes sense to me for this not to be a method on the model class, and instead be something users do on their own prior to dataset concatenation or alignment.

@romain-lopez any thoughts on this?

Hey Adam,
I still think what I suggested in https://github.com/YosefLab/scVI/pull/495#issuecomment-556097875 could be a nice behavior.
The changes would be:

[ ] Remove the .upper() in the different dataset files
[ ] Add a flag case_sensitive=False in dataset.reorder_genes and in the constructors of the datasets accepting a genes_to_keep argument; such as CortexDataset or FrontalCortexDropseqDataset in loom.py
[ ] Change the behavior of the previous functions to work accordingly: if case_sensitive==True then it should be the same code -- since now upper was removed -- and else we should do .upper only in the look_up dictionaries we use to match the gene names (see code below).

This way, gene names will never be changed but the gene_to_keep arguments will have a very handful argument to be case insensitive.
https://github.com/YosefLab/scVI/blob/c1299048cd25398d6df65547f7dcf3fba52a4d7d/scvi/dataset/dataset.py#L937-L938

I disagree that we should allow the code to be invariant to gene name capitalization as it seems like bad practice.
We could just throw a warning in populate from datasets if there are very few shared gene names suggesting the user checks gene names themselves .

@adamgayoso I let you decide since you know more about the casual usage of gene names.
However the uppercasing was introduced in the repo to avoid problems with case sensitive matching. With your approach, if two datasets have different case (notably some dataset with capitalized or uppercased genes), we will have to upper case the gene names with make_gene_names_upper although it was specifically the thing we wanted to avoid. Thus I still think a flag case_sensitive: bool is more appropriate since it will fix the original issue. Or rather use make_gene_names_lower since upper seems to be a problem.
In the end, the most important is to get the PR merged since some people need this to be fixed fast. Thanks for your changes and feel free to change at least the make_gene_lower!

@ANazaret I'll change it to lower.
I think the solution proposed in the code is more generally suitable. Users will only have to make gene names have the same case in advanced usage situations.

I think we should at the very least do two things. (A) let the user know whether we changed gene names for merging or any other reasons, that should be done via logging (B) that should not be done by default.
Achille's rule might not be perfectly suitable since even once you merged the gene names in a case unsensititive way, you need to pick one of the two gene names. So Adam's solution is surely arbitrary but not too bad as long as (A) we inform the user and (B) this is done exceptionally and not the default behavior


RDF4JSUT [00001] - Add tests for repo generation use cases

Use cases should include:

Native Sail

Create new repository only if repository does not exist

Overwrite existing repository

Lucene Sail

Create new repository only if repository does not exist

Overwrite existing repository

Fixed in commit 2a82445ff8a452e9cae333b9a0e3b0223b95034e branch test_rdf4jsut_00001.


create deluxe.scss

Hi! What is your PR about?

[x] Add or edit a skin
[ ] Add a custom domain for your space
[ ] Add an alias to migrate your space

To create your skin will also need to list your skin on this file: https://github.com/snapshot-labs/snapshot-spaces/blob/master/skins/index.js

yes we will just use the default theme thanks


OSError: [Errno 24] Too many open files

error if too many files to concatenate. In this example: 384 files.
data: errsm_40/t2 --> split them along y, then concatenate.
sct_image -i tmp.anat_orient_fit_Z*.nii -o test.nii.gz -concat z
Traceback (most recent call last):
  File "/Users/julien/code/spinalcordtoolbox/bin/sct_image", line 474, in <module>
    main()
  File "/Users/julien/code/spinalcordtoolbox/bin/sct_image", line 120, in main
    im_in = [Image(fn) for fn in fname_in]
  File "/Users/julien/code/spinalcordtoolbox/scripts/msct_image.py", line 44, in __init__
    self.loadFromPath(param, verbose)
  File "/Users/julien/code/spinalcordtoolbox/scripts/msct_image.py", line 100, in loadFromPath
    self.orientation = get_orientation(path, filename=True)
  File "/Users/julien/code/spinalcordtoolbox/scripts/sct_image.py", line 420, in get_orientation
    status, output = run('isct_orientation3d -i '+im+' -get ', 0)
  File "/Users/julien/code/spinalcordtoolbox/scripts/sct_utils.py", line 80, in run
    process = subprocess.Popen(cmd, shell=True, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
  File "/Users/julien/miniconda/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/Users/julien/miniconda/lib/python2.7/subprocess.py", line 1223, in _execute_child
    errpipe_read, errpipe_write = self.pipe_cloexec()
  File "/Users/julien/miniconda/lib/python2.7/subprocess.py", line 1175, in pipe_cloexec
    r, w = os.pipe()
OSError: [Errno 24] Too many open files

also called in sct_flatten_sagittal

create a loop and open/close each file within the loop

I've been checking my wrapper with this function to kill two bird with one stone, and it seems that the bug is pretty deep in nibabel. The  get_unscaled() method of the ArrayProxy() class in nibabel/arrayproxy.py line 128.
   def get_unscaled(self):
        ''' Read of data from file

        This is an optional part of the proxy API
        '''
        with BinOpener(self.file_like) as fileobj:
            raw_data = array_from_file(self._shape,
                                       self._dtype,
                                       fileobj,
                                       offset=self._offset,
                                       order=self.order,
                                       mmap=self._mmap)
        return raw_data```

The fileobj is never pour out of scope, so the file does not close. And file keeps opening.

I manage to fix it in nibable... I can open a ticket with them.

The fix
      def get_unscaled(self):
           ''' Read of data from file

           This is an optional part of the proxy API
           '''
           with BinOpener(self.file_like) as fileobj:
               raw_data = array_from_file(self._shape,
                                          self._dtype,
                                          fileobj,
                                          offset=self._offset,
                                          order=self.order,
                                          mmap=self._mmap)
+++        fileobj.close()
           return raw_data```

Fixed on branch ``ìssue_715```. Tested split+concat along Y on errsm_40, result is identical to original image :


Add makinarocks link to external Adds-on section

fix #3441
MakinaRocks Link is a development tool for building a pipeline for ML workflow. Link Pipeline can be integrated with Kubeflow Pipeline, so I add it to external-add-ons section.
You can check out how it can be integrated -> Export/Import Kubeflow Pipeline

Can anyone visit this PR, please?

/close
due to page and content does not currently exist.
cc @hunhoon21  @hbelmiro


[IPAM] core tests

Description
This PR includes the test for the IPAM core package

/test

/rebase test=true

/rebase test=true


Update release instructions with more details.

@jameslinjl can you take a quick look at this documentation update?

@isabelle-dr this doc update has an LGTM from James but needs an official review from someone with commit privileges (and I can't review my own PR).  Any chance you could sign off?


Enable CI tests for Mamba functional model

This adds all of our unit tests to the nightly pipeline. Some of these tests are quick (<50ms) so if we can add them to a different pipeline that will run more frequently, let me know.
We have only tested on WH, so I did not enable the GS tests. I see that some other models use @skip_for_grayskull decorator instead of in the run_tests.sh script - let me know if that is preferred.

Closing since these changes will be added in #6744