-
Notifications
You must be signed in to change notification settings - Fork 22
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
FRI 2.6 #33
Comments
hi @matija9 and thank you for creating this issue. Apologize the late response, I wasn't notified. Please do feel free to tag me! There is now a branch |
Hi @mhubii thanks for the response and for opening the branch. |
I'll keep this open until this is added to tests etc |
I have just now gotten to installing lbr_fri_ros2_stack with fri 2.6 and I get an error that file: I didn't tag you so as not to disturb you during the holidays and there is no big rush with it. |
hi @matija9 and apologize the late response. Please do feel free to tag. Could you post your installation steps? |
hi @mhubii, now i have to apologize for the late response. I had ros-dev-tools installed previously, then i tried running following set of commands: source /opt/ros/humble/setup.bash
export FRI_CLIENT_VERSION=2.6
mkdir -p lbr-stack-26/src && cd lbr-stack-26
vcs import src --input https://raw.githubusercontent.com/lbr-stack/lbr_fri_ros2_stack/humble/lbr_fri_ros2_stack/repos-fri-${FRI_CLIENT_VERSION}.yaml
rosdep install --from-paths src -i -r -y While executing these commands i get Then i have tried to manually clone all of the required repos, but when i tried building the workspace i got the error stating there s wrong version of FRI and that it expected version 1.15, but that was easily fixed by adjusting CMakeLists. So now it have all been installed , but sadly my robot has some hardware issues so it will be a few weeks before i can test if it all works for real. |
hi @matija9, the issue is the following: The Here are steps you can do manually:
source /opt/ros/humble/setup.bash
export FRI_CLIENT_VERSION=1.15
mkdir -p lbr-stack-26/src && cd lbr-stack-26
vcs import src --input https://raw.githubusercontent.com/lbr-stack/lbr_fri_ros2_stack/humble/lbr_fri_ros2_stack/repos-fri-${FRI_CLIENT_VERSION}.yaml
git --git-dir=src/fri/.git --work-tree=src/fri checkout fri-2.6
rosdep install --from-paths src -i -r -y
colcon build --symlink-install I hope this fixes your issue. If you could create a pull request that adds the Please note that |
@mhubii thanks for the response. I have managed to download and install everything, but as i mentioned in previous comment there was also a problem with And i can create a pull request for |
hi @matija9, yes, KUKA is a little sloppy with their versioning. As a client, it is hard to track what or whether something changed, that's why we decided to just treat the FRI SDK as a black box. If your files list |
Hi @matija9, sorry for the delay. I've now had the chance to have a look at the Line 62 in 5811942
You can quickly verify this through git clone [email protected]:lbr-stack/fri.git -b fri-2.6 && \
cd fri && \
cmake -B build This will throw CMake Error at CMakeLists.txt:63 (message):
Expected FRIClient version 1.15, found: 2.5 Since this is a client to the SDK, it might not be ideal to support KUKA's shortcoming here. Let me know what you think. Maybe in your PR here lbr-stack/lbr_fri_ros2_stack#248, one may just point to |
Hi @mhubii, in #36 i have changed that regular expression to look for version 2.5 so it can build properly. I know that is not the best solution but it works with KUKA's broken versioning.
You mean just use SDK from 2.5 for 2.6 as well? I am not that sure about it because there still could be some meaningful changes even tho version number in the file is same. I can test running the robot with "wrong" version of FRI. I can get back to you next week with the results. Also i will try to get a hold of someone else with the robot running version 2.6 if i can to check if i maybe got a broken version of FRI on my robot. |
thanks for investigating all this! I just compared the contents of the zip files in I agree that this is very confusing to users, and wonder how one could avoid the confusion. |
As per readme, please create branch for FRI version 2.6.
The text was updated successfully, but these errors were encountered: