Skip to content
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

fix issue #810 #863

Open
wants to merge 2 commits into
base: rolling
Choose a base branch
from
Open

fix issue #810 #863

wants to merge 2 commits into from

Conversation

xlla
Copy link

@xlla xlla commented May 10, 2022

to solve the error

[rviz2-3] [ERROR] [1652204061.476182910] [rviz2]: ItemIdentityException: Can not find codec for 'tif' image format.
[rviz2-3] Supported formats are: bmp dds gif hdr jpeg jpg ktx pgm pic pkm png ppm psd tga. in Codec::getCodec at /Users/xlla/ros2_foxy/build/rviz_ogre_vendor/ogre-v1.12.1-prefix/src/ogre-v1.12.1/OgreMain/src/OgreCodec.cpp (line 66)

Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First of all, thanks for starting this.

Unfortunately, this change on its own is not enough. In particular, we also need to make sure that libfreeimage-dev is installed on the system before we use this, otherwise we get errors like:

Could not load dynamic library install/rviz_ogre_vendor/opt/rviz_ogre_vendor/lib/OGRE/Codec_FreeImage

To do that, we need to add <depend>libfreeimage-dev</depend> to rviz_ogre_vendor, and we also need to add libfreeimage to the rosdep database. I've opened a pull request for the latter at ros/rosdistro#33403 .

We also need to test this out on Windows to make sure that it works there. Finally, we should target this to the ros2 branch first, as we want to get this onto Rolling, and then think about backporting to Humble.

@clalancette clalancette self-assigned this May 31, 2022
@clalancette
Copy link
Contributor

To do that, we need to add <depend>libfreeimage-dev</depend> to rviz_ogre_vendor, and we also need to add libfreeimage to the rosdep database. I've opened a pull request for the latter at ros/rosdistro#33403 .

All right, that PR is in. I'm going to re-target this to the ros2 branch, and also add those rosdep keys to this PR. That should solve the problem for Linux.

For Windows, I think we'll have to add freeimage as part of the build process in rviz_ogre_vendor. If we do that, that should make this work on all platforms.

@clalancette clalancette changed the base branch from humble to ros2 June 1, 2022 18:53
Signed-off-by: xlla <xllacyx@hotmail.com>
That way freeimage is available and OGRE support for it is
built when building this vendor package.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@audrow audrow changed the base branch from ros2 to rolling June 28, 2022 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants