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

Apartment: xacro file must be compiled into URDF #106

Open
daniel86 opened this issue Sep 27, 2022 · 1 comment
Open

Apartment: xacro file must be compiled into URDF #106

daniel86 opened this issue Sep 27, 2022 · 1 comment

Comments

@daniel86
Copy link
Contributor

daniel86 commented Sep 27, 2022

the urdf library used by knowrob cannot handle xacro statements, thus the apartment.urdf file must be compiled by xacro.
either the file should be uploaded into github, or we do it compile-time.

side-note: shouldn't have the apartment.urdf file have the ending "xacro"?

@daniel86 daniel86 changed the title xacro file must be compiled into URDF Apartment: xacro file must be compiled into URDF Sep 28, 2022
HoangGiang93 added a commit that referenced this issue Oct 13, 2022
@HoangGiang93
Copy link
Contributor

HoangGiang93 commented Oct 13, 2022

the urdf library used by knowrob cannot handle xacro statements, thus the apartment.urdf file must be compiled by xacro. either the file should be uploaded into github, or we do it compile-time.

side-note: shouldn't have the apartment.urdf file have the ending "xacro"?

I changed the name from apartment.urdf to furnitures.xacro, because it contains a part of apartment.
Then I created a new apartment.xacro, which contains furnitures.xacro and kitchen.xacro.
You can use the apartment.xacro to generate apartment.urdf.
I also changed the code in CMakeLists.txt.

image

HoangGiang93 added a commit that referenced this issue Oct 13, 2022
HoangGiang93 added a commit that referenced this issue Oct 13, 2022
This reverts commit 1a6a800.
HoangGiang93 added a commit that referenced this issue Oct 13, 2022
HoangGiang93 added a commit that referenced this issue Oct 13, 2022
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

No branches or pull requests

2 participants