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

add option to change package:// to model:// when loading urdf file #288

Merged
merged 1 commit into from
Apr 27, 2016

Conversation

k-okada
Copy link
Contributor

@k-okada k-okada commented Jan 1, 2015

this feature is required if we want to use gzweb, but ofcourse you have to set GAZEBO_MODEL_PATH correctly

@hsu
Copy link
Collaborator

hsu commented Mar 11, 2015

@k-okada do you have an example? tutorial? thanks.

@130s
Copy link

130s commented Apr 15, 2015

@hsu
Copy link
Collaborator

hsu commented Jan 20, 2016

+1

@scpeters scpeters added this to the kinetic release milestone Apr 25, 2016
@hsu hsu merged commit 8c910da into ros-simulation:indigo-devel Apr 27, 2016
@@ -243,6 +248,9 @@ class SpawnModel():
print "Error: user specified param or filename is an empty string"
sys.exit(0)

if self.package_to_model:
model_xml = re.sub("<\s*mesh\s+filename\s*=\s*\"package:/?/?","<mesh filename=\"model://", model_xml)
Copy link
Member

Choose a reason for hiding this comment

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

this only works for attributes with double quotes ". I think it should be updated to handle single quotes ' as well

Copy link
Collaborator

Choose a reason for hiding this comment

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

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.

4 participants