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

Make output directories of RobotImporter unique #394

Merged

Conversation

lewandowskikacper
Copy link
Contributor

@lewandowskikacper lewandowskikacper commented Jul 10, 2023

Added Uuid of parameters passed to XACRO to the name name of RobotImporter's output directory.
This makes the directory unique for each robot imported from the same xacro file.
Addresses #392.

Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
@lewandowskikacper lewandowskikacper requested review from a team as code owners July 10, 2023 11:17
Copy link
Contributor

@michalpelka michalpelka left a comment

Choose a reason for hiding this comment

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

Please adjust changes to CopyAssetForURDFAndCreateAssetMap

Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
lewandowskikacper and others added 5 commits July 11, 2023 09:36
Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
Signed-off-by: Kacper Lewandowski <kacper.lewandowski@robotec.ai>
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.

5 participants