Skip to content

[Noetic] Update RViz import#60

Merged
wjwwood merged 1 commit into
ros-visualization:kinetic-develfrom
sloretz:update_rviz_import
May 13, 2020
Merged

[Noetic] Update RViz import#60
wjwwood merged 1 commit into
ros-visualization:kinetic-develfrom
sloretz:update_rviz_import

Conversation

@sloretz

@sloretz sloretz commented May 13, 2020

Copy link
Copy Markdown

This updates the import of the rviz bindings as described here in the Noetic Migration guide. Without this the script fails to find rviz.VisualizationFrame(). This change must be merged into a to-be-created noetic-devel branch, and not kinetic-devel.

I'm currently unable to verify the rest of the script works because I'm unable to launch RViz in a container, but it may just be a problem with my container setup.

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz sloretz requested a review from wjwwood May 13, 2020 03:06
@wjwwood wjwwood merged commit 60eba95 into ros-visualization:kinetic-devel May 13, 2020
@sloretz

sloretz commented May 13, 2020

Copy link
Copy Markdown
Author

@wjwwood oops it looks like this one got merged into kinetic-devel, but it should really go in a separate noetic-devel branch.

@wjwwood

wjwwood commented May 13, 2020

Copy link
Copy Markdown
Member

You’re right. I’ll fix that today. I didn’t release into kinetic or melodic yet though.

wjwwood added a commit that referenced this pull request May 13, 2020
@wjwwood

wjwwood commented May 13, 2020

Copy link
Copy Markdown
Member

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.

2 participants