Fix compatibility with roslibpy 2.0.0 by adding upper bound constraint #453
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.
Problem
The recent release of
roslibpy 2.0.0
introduced breaking changes to the API, specifically moving theactionlib
module fromroslibpy.actionlib
toroslibpy.ros1.actionlib
. This caused import errors incompas_fab
:Solution
This PR adds an upper bound constraint to the
roslibpy
dependency to prevent installation of version 2.0.0 and above:This ensures compatibility with the current codebase until
compas_fab
can be updated to support roslibpy 2.x and ROS 2 in a future release.Changes
requirements.txt
to constrainroslibpy < 2
CHANGELOG.md
documenting the fixTesting
Related Issues
Fixes the issue reported where pytest imports were failing due to the roslibpy 2.0.0 breaking changes.
Future Work
A future release will need to update the codebase to support roslibpy 2.x, which includes support for ROS 2. This will require updating import statements and potentially other API changes throughout the ROS backend.
Original prompt
Fixes #451
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.