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 #6: No robot model when using display.launch (again) #7

Merged
merged 3 commits into from Jan 9, 2014

Conversation

gavanderhoorn
Copy link
Contributor

This pull request should actually fix the issues. I think the proposed file and directory layout is in line with other packages containing URDFs and launch files.

Doing it this way does require us updating the tutorials, as the URDFs are now in the urdf subdirectory.

PS: I left the urdf.vcg file in, although AFAIK those are not used anymore (old RViz config file?).

@gavanderhoorn
Copy link
Contributor Author

Ping?

isucan added a commit that referenced this pull request Jan 9, 2014
…stall

Fix issue #6: No robot model when using display.launch (again)
@isucan isucan merged commit 766363b into ros:master Jan 9, 2014
@gavanderhoorn
Copy link
Contributor Author

Thanks. Should we update the tutorial pages now, or when this is released?

@gavanderhoorn gavanderhoorn deleted the issue6_source_vs_devel_vs_install branch January 9, 2014 16:08
@isucan
Copy link
Contributor

isucan commented Jan 9, 2014

Can you update it now? I will trigger a release now for Hydro. Should I release for Groovy as well?

@gavanderhoorn
Copy link
Contributor Author

I can update them.

Should I release for Groovy as well?

Well AFAIK the package is currently broken in both Hydro and Groovy, so I guess it should be rereleased in both?

@gavanderhoorn
Copy link
Contributor Author

I've updated the wiki pages I could find. There are still some bits that seem outdated (references to urdf_tools package etc) though.

@isucan
Copy link
Contributor

isucan commented Jan 9, 2014

Packages released for both hydro & groovy

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