-
Notifications
You must be signed in to change notification settings - Fork 62
[nextage_tutorials] Add current setup of jsk Nextage Open. #627
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
[nextage_tutorials] Add current setup of jsk Nextage Open. #627
Conversation
| @@ -0,0 +1,18 @@ | |||
| (defun nextage-collision-check () | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In most cases, <robot>-utils.l defines functions by overwriting/adding methods to the existing robot class:
https://github.com/jsk-ros-pkg/jsk_robot/blob/b193dc5a09d8e73aa1fb1100297516404238d4aa/jsk_fetch_robot/fetcheus/fetch-utils.l
https://github.com/jsk-ros-pkg/jsk_robot/blob/b193dc5a09d8e73aa1fb1100297516404238d4aa/jsk_panda_robot/panda_eus/euslisp/panda-utils.l
Why don't you use this method?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why don't you overwrite :collision-check-pairs like jsk-ros-pkg/jsk_robot#1026?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WIP.
This is temporary and the easiest way to just test . This is why I said this PR is Draft. When I overwrite the robot interface, Then I change this PR from Draft to Open.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed nextage-utils.l by overwriting :self-collision-check in NextageOpen-robot.
I also edited nextage-interface.l to call :start-grasp and :stop-grasp from ri.
| <exec_depend>euscollada</exec_depend> | ||
| <exec_depend>nextage_description</exec_depend> | ||
| <exec_depend>nextage_gazebo</exec_depend> | ||
| <!--<exec_depend>nextage_gazebo</exec_depend>--> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you comment out this?
I think rosdep install displays an error but we can skip it.
Is there any critical error?
(Perhaps I just forgot.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, we can skip when running rosdep insall, but when we run catkin build, some dependencies problem cannnot be resolved. You said that currently this problem cannot be solved, so I was told to comment out this to pass catkin build.
[build] Found 146 packages in 0.0 seconds.
[build] Updating package table.
Starting >>> nextage_description
Starting >>> openrtm_tools
Starting >>> rtmbuild
Finished <<< nextage_description [ 0.1 seconds ]
Finished <<< rtmbuild [ 0.1 seconds ]
_______________________________________________________________________________
Errors << openrtm_tools:cmake /home/leus/catkin_ws/logs/openrtm_tools/build.cmake.007.log
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
Compatibility with CMake < 2.8.12 will be removed from a future version of
CMake.
Update the VERSION argument <min> value or use a ...<max> suffix to tell
CMake that the project does not need compatibility with older versions.
CMake Warning (dev) at CMakeLists.txt:2 (project):
Policy CMP0048 is not set: project() command manages VERSION variables.
Run "cmake --help-policy CMP0048" for policy details. Use the cmake_policy
command to set the policy and suppress this warning.
The following variable(s) would be set to empty:
CMAKE_PROJECT_VERSION
CMAKE_PROJECT_VERSION_MAJOR
CMAKE_PROJECT_VERSION_MINOR
CMAKE_PROJECT_VERSION_PATCH
This warning is for project developers. Use -Wno-dev to suppress it.
CMake Error at /usr/share/cmake-3.22/Modules/FindPkgConfig.cmake:603 (message):
A required package was not found
Call Stack (most recent call first):
/usr/share/cmake-3.22/Modules/FindPkgConfig.cmake:825 (_pkg_check_modules_internal)
CMakeLists.txt:54 (pkg_check_modules)
cd /home/leus/catkin_ws/build/openrtm_tools; catkin build --get-env openrtm_tools | catkin env -si /usr/bin/cmake /home/leus/catkin_ws/src/rtmros/rtmros_common/openrtm_tools --no-warn-unused-cli -DCATKIN_DEVEL_PREFIX=/home/leus/catkin_ws/devel/.private/openrtm_tools -DCMAKE_INSTALL_PREFIX=/home/leus/catkin_ws/install; cd -
...............................................................................
Failed << openrtm_tools:cmake [ Exited with code 1 ]
Failed <<< openrtm_tools [ 0.7 seconds ]
Abandoned <<< hrpsys_tools [ Unrelated job failed ]
Abandoned <<< nextage_tutorials [ Unrelated job failed ]
Abandoned <<< hrpsys_ros_bridge [ Unrelated job failed ]
Abandoned <<< hironx_ros_bridge [ Unrelated job failed ]
Abandoned <<< hironx_moveit_config [ Unrelated job failed ]
Abandoned <<< nextage_ros_bridge [ Unrelated job failed ]
Abandoned <<< nextage_moveit_config [ Unrelated job failed ]
Abandoned <<< nextage_gazebo [ Unrelated job failed ]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I attach the full log. Missing package is openrtm-aist.
leus@leus-ThinkPad-T14s-Gen-5:~/catkin_ws/logs/openrtm_tools$ cat build.cmake.log
Not searching for unused variables given on the command line.
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
Compatibility with CMake < 2.8.12 will be removed from a future version of
CMake.
Update the VERSION argument <min> value or use a ...<max> suffix to tell
CMake that the project does not need compatibility with older versions.
CMake Warning (dev) at CMakeLists.txt:2 (project):
Policy CMP0048 is not set: project() command manages VERSION variables.
Run "cmake --help-policy CMP0048" for policy details. Use the cmake_policy
command to set the policy and suppress this warning.
The following variable(s) would be set to empty:
CMAKE_PROJECT_VERSION
CMAKE_PROJECT_VERSION_MAJOR
CMAKE_PROJECT_VERSION_MINOR
CMAKE_PROJECT_VERSION_PATCH
This warning is for project developers. Use -Wno-dev to suppress it.
-- Using CATKIN_DEVEL_PREFIX: /home/leus/catkin_ws/devel/.private/openrtm_tools
-- Using CMAKE_PREFIX_PATH: /home/leus/catkin_ws/devel;/opt/ros/one
-- This workspace overlays: /home/leus/catkin_ws/devel;/opt/ros/one
-- Using PYTHON_EXECUTABLE: /usr/bin/python3
-- Using Debian Python package layout
-- Using empy: /usr/bin/empy
-- Using CATKIN_ENABLE_TESTING: ON
-- Call enable_testing()
-- Using CATKIN_TEST_RESULTS_DIR: /home/leus/catkin_ws/build/openrtm_tools/test_results
-- Forcing gtest/gmock from source, though one was otherwise available.
-- Found gtest sources under '/usr/src/googletest': gtests will be built
-- Found gmock sources under '/usr/src/googletest': gmock will be built
-- Using Python nosetests: /usr/bin/nosetests3
-- catkin 0.8.12
-- BUILD_SHARED_LIBS is on
-- Checking for module 'openrtm-aist'
-- No package 'openrtm-aist' found
CMake Error at /usr/share/cmake-3.22/Modules/FindPkgConfig.cmake:603 (message):
A required package was not found
Call Stack (most recent call first):
/usr/share/cmake-3.22/Modules/FindPkgConfig.cmake:825 (_pkg_check_modules_internal)
CMakeLists.txt:54 (pkg_check_modules)
-- Configuring incomplete, errors occurred!
See also "/home/leus/catkin_ws/build/openrtm_tools/CMakeFiles/CMakeOutput.log".
f6af72f to
1eb4b97
Compare
…a0bResetPose length
…ween XYZ and color
On Ubuntu Focal, "/usr/bin/env python" cannot be executed unless python-is-python2/3 is installed
c40012c to
41b5881
Compare
…real robot and use each limb controller when using gazebo simulator
| (when on-gazebo-ros-control | ||
| (ros::ros-warn "Found Gazebo/ros_control environment")) | ||
| ) | ||
| (when on-gazebo-ros-control |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Do not read the following until you finish your bachelor thesis)
This when does not work as you expect:
1.irteusgl$ (setq var t)
t
2.irteusgl$ (defclass test :super object :slots ())
test
3.irteusgl$ (defmethod test (:init ()) (when var (:test ())))
test
4.irteusgl$ (setq test-ins (instance test :init))
#<test #X560f20ad91a8>
5.irteusgl$ (send test-ins :methods)
(:init when :prin1 :warning :error :slots :methods :super :get-val :set-val)
Therefore, hironxjsk-interface defines and uses special :gazebo-ros-controller:
| (on-gazebo-ros-control :gazebo-ros-controller) |
rtmros_tutorials/hrpsys_ros_bridge_tutorials/euslisp/hironxjsk-interface.l
Lines 62 to 67 in 221301d
| (:gazebo-ros-controller () | |
| (append | |
| (send self :rarm-controller) | |
| (send self :larm-controller) | |
| (send self :head-controller) | |
| (send self :torso-controller))) |
Or you can use macro, one of the biggest advantages of Lisp (but readability may be decreased):
https://github.com/start-jsk/rtmros_common/blob/57331a5179d0d10947ff6f58eae744aa8c0be22a/hrpsys_ros_bridge/euslisp/rtm-ros-robot-interface.l#L303-L322
Currently, systems for robots in JSK related to hrpsys is managed and maintained in this repository (not in jsk_robot)
This PR adds current Nextage Open Setup in JSK to nextage_tutorials, as following the "hironxjsk". I use the "jsk-nextage" namespace to distinguish between the origina nextage system and jsk nextage system.
Summary
.gitignoreto unstage the .dae and .l files which is generated when catkin build.To overwrite the TF, this node uses the name hrpsys_state_publisher same as the one of publisher in the robot's PC. As a result, the original node is killed and the new one overwrites it.
serial_noto distinguish between l515 and d405. Because I designate the serial no in this file, we do not have to prepare udev to distinguish among them as for realsense.