Repository navigation
Release/v2.1 - #36
Merged
Merged
Release/v2.1#36
Conversation
ZhipengZhang-flexiv
requested review from
bsoe-flexiv,
munseng-flexiv,
xyyeh-flexiv and
zfquek-flexiv
July 29, 2026 04:07
There was a problem hiding this comment.
Pull request overview
Updates Flexiv TDK to the v2.1 release line, aligning build/install tooling and public teleoperation APIs with flexiv_rdk 2.1 and refreshing bundled release artifacts (hashes, docs, examples).
Changes:
- Bump TDK versioning to 2.1 across CMake/README/Doxygen and require
flexiv_rdk2.1. - Update teleoperation public headers and examples (C++/Python): introduce
NetworkCfgStd, addJointGroupparameters, and remove some legacy LAN/WAN/Jazzy-related options/APIs. - Remove joint-teleop LAN headers/examples and ROS2 Jazzy library hashes; refresh release SHA256 sums.
Reviewed changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| thirdparty/scripts/install_flexiv_rdk.sh | Updates RDK tag to v2.1 and makes existing clones fetch/checkout the requested tag. |
| thirdparty/build_and_install_dependencies.sh | Removes ROS2 Jazzy flag handling from dependency installer script. |
| README.md | Updates version badge to 2.1. |
| lib/libflexiv_tdk.x86_64-linux-gnu.ros2-jazzy.a.sha256 | Removes Jazzy-specific checksum file. |
| lib/libflexiv_tdk.x86_64-linux-gnu.a.sha256 | Updates checksum for the x86_64 static library artifact. |
| lib/libflexiv_tdk.aarch64-linux-gnu.ros2-jazzy.a.sha256 | Removes Jazzy-specific checksum file. |
| lib/libflexiv_tdk.aarch64-linux-gnu.a.sha256 | Updates checksum for the aarch64 static library artifact. |
| include/flexiv/tdk/transparent_cartesian_teleop_wan.hpp | Updates WAN teleop API/docs: NetworkCfgStd, JointGroup parameters, API removals/reordering. |
| include/flexiv/tdk/transparent_cartesian_teleop_lan.hpp | Updates LAN teleop API/docs: removes whitelist ctor arg, adds JointGroup parameters. |
| include/flexiv/tdk/joint_teleop_lan.hpp | Removes joint-space LAN teleop public header. |
| include/flexiv/tdk/device_teleop_lan.hpp | Updates device-teleop LAN API to include JointGroup parameters and removes whitelist ctor arg. |
| include/flexiv/tdk/data.hpp | Introduces NetworkCfgStd, adjusts IO port constant, and updates networking documentation. |
| example/transparent_cartesian_teleop_wan.cpp | Updates WAN C++ example to new API (NetworkCfgStd, JointGroup). |
| example/transparent_cartesian_teleop_lan.cpp | Updates LAN C++ example to new API (JointGroup, ctor signature). |
| example/joint_teleop_under_lan.cpp | Removes joint-space LAN teleop C++ example. |
| example/CMakeLists.txt | Drops the removed joint-space LAN example from the build list. |
| example_py/transparent_cartesian_teleop_wan.py | Updates WAN Python example to new API (NetworkCfgStd, JointGroup) and interface whitelist semantics. |
| example_py/transparent_cartesian_teleop_lan.py | Updates LAN Python example to new API (JointGroup, ctor signature) and removes HomeAll menu actions. |
| example_py/joint_teleop_under_lan.py | Removes joint-space LAN teleop Python example. |
| docs/doxygen/Doxyfile.in | Updates Doxygen project number to 2.1. |
| CMakeLists.txt | Bumps project version, removes Jazzy option, updates flexiv_rdk requirement to 2.1 EXACT, and keeps release download logic. |
| cmake/flexiv_tdk-config.cmake.in | Updates exported config dependency to flexiv_rdk 2.1. |
Comments suppressed due to low confidence (3)
include/flexiv/tdk/transparent_cartesian_teleop_lan.hpp:268
- Typo in API documentation: "fuction" -> "function".
* @warning This fuction is less efficient than the other overloaded one as additional runtime
* memory allocation and data copying are performed.
include/flexiv/tdk/transparent_cartesian_teleop_lan.hpp:24
- Grammar in the class note is off ("robot which operated" / "robot interacts with"). This is user-facing API documentation, so it’s worth fixing for clarity.
* @note In the documentation of this class, "leader robot" refers to the robot which operated by a
* human operator during teleoperation; "follower robot" refers to the robot interacts with
* workpieces.
include/flexiv/tdk/transparent_cartesian_teleop_wan.hpp:295
- Typo in API documentation: "fuction" -> "function".
* @warning This fuction is less efficient than the other overloaded one as additional runtime
* memory allocation and data copying are performed.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
added 2 commits
July 29, 2026 13:56
pzhu-flexiv
approved these changes
Jul 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.