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

use PrimitiveAssets Gem to visualize primitives #485

Merged

Conversation

jhanca-robotecai
Copy link
Contributor

Fix #478

PrimitiveAssets Gem is used to visualize primitives. The change was tested on simple URDF containing all four types of visuals.
Screenshot from 2023-08-30 13-56-18

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
@jhanca-robotecai jhanca-robotecai requested review from a team as code owners August 30, 2023 12:05
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
@jhanca-robotecai jhanca-robotecai marked this pull request as draft August 30, 2023 12:29
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
@jhanca-robotecai jhanca-robotecai marked this pull request as ready for review August 30, 2023 14:51
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
@jhanca-robotecai
Copy link
Contributor Author

Code review comments were applied, and one more error was spotted. In particular, sphere and cylinder objects in Primitives Gem have a diameter of 1 (I expected it to be a radius).

After change:
Screenshot from 2023-09-01 14-57-10

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
@adamdbrw adamdbrw merged commit d572625 into o3de:development Sep 5, 2023
mbalfour-amzn pushed a commit to aws-lumberyard-dev/o3de-extras that referenced this pull request Sep 7, 2023
Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
mbalfour-amzn added a commit that referenced this pull request Sep 14, 2023
…remappings (#505)

* First version of adding configurable prefix lookups and substitutions.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Tests/UrdfParserTest.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/SdfAssetBuilder/SdfAssetBuilderSettings.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Improved asset path resolution logic and unit tests.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Add SDFormat importer hooks (#453)

Add sensors SDF importer hooks

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* use PrimitiveAssets Gem to visualize primitives (#485)

Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* Warehouse automation gem (#440)

Created the warehouse automation gem, adjusted to review

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>

* Lidar component refactor (#463)

Lidar Refactor

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>

* Added presentation of SDF messages (#491)

* Added presentation of SDF messages

---------
Signed-off-by: Michał Pełka <michalpelka@gmail.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>

* Fix missing include.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Fix resolution of file:// prefix.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Change string to Path.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Changed PathResolver to get read in wholesale from setreg file.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback - changed file select dir to default to last-used file.
Also changes file select dir to match anything typed into the text box.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Partial revert from path back to string.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Second half of path to string revert.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Added all Path Resolver options to setreg for visibility.
Improved SetFindCallback to use the full set of path resolution options.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

---------

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>
Co-authored-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>
Co-authored-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka pushed a commit to RobotecAI/o3de-extras that referenced this pull request Sep 25, 2023
Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
michalpelka added a commit to RobotecAI/o3de-extras that referenced this pull request Sep 25, 2023
…remappings (o3de#505)

* First version of adding configurable prefix lookups and substitutions.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Tests/UrdfParserTest.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/SdfAssetBuilder/SdfAssetBuilderSettings.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Improved asset path resolution logic and unit tests.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Add SDFormat importer hooks (o3de#453)

Add sensors SDF importer hooks

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* use PrimitiveAssets Gem to visualize primitives (o3de#485)

Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* Warehouse automation gem (o3de#440)

Created the warehouse automation gem, adjusted to review

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>

* Lidar component refactor (o3de#463)

Lidar Refactor

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>

* Added presentation of SDF messages (o3de#491)

* Added presentation of SDF messages

---------
Signed-off-by: Michał Pełka <michalpelka@gmail.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>

* Fix missing include.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Fix resolution of file:// prefix.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Change string to Path.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Changed PathResolver to get read in wholesale from setreg file.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback - changed file select dir to default to last-used file.
Also changes file select dir to match anything typed into the text box.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Partial revert from path back to string.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Second half of path to string revert.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Added all Path Resolver options to setreg for visibility.
Improved SetFindCallback to use the full set of path resolution options.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

---------

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>
Co-authored-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>
Co-authored-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka added a commit to RobotecAI/o3de-extras that referenced this pull request Sep 25, 2023
…remappings (o3de#505)

* First version of adding configurable prefix lookups and substitutions.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Tests/UrdfParserTest.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/SdfAssetBuilder/SdfAssetBuilderSettings.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Improved asset path resolution logic and unit tests.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Add SDFormat importer hooks (o3de#453)

Add sensors SDF importer hooks

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* use PrimitiveAssets Gem to visualize primitives (o3de#485)

Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* Warehouse automation gem (o3de#440)

Created the warehouse automation gem, adjusted to review

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>

* Lidar component refactor (o3de#463)

Lidar Refactor

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>

* Added presentation of SDF messages (o3de#491)

* Added presentation of SDF messages

---------
Signed-off-by: Michał Pełka <michalpelka@gmail.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>

* Fix missing include.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Fix resolution of file:// prefix.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Change string to Path.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Changed PathResolver to get read in wholesale from setreg file.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback - changed file select dir to default to last-used file.
Also changes file select dir to match anything typed into the text box.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Partial revert from path back to string.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Second half of path to string revert.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Added all Path Resolver options to setreg for visibility.
Improved SetFindCallback to use the full set of path resolution options.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

---------

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>
Co-authored-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>
Co-authored-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka pushed a commit to RobotecAI/o3de-extras that referenced this pull request Sep 25, 2023
Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka added a commit to RobotecAI/o3de-extras that referenced this pull request Sep 25, 2023
…remappings (o3de#505)

* First version of adding configurable prefix lookups and substitutions.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Tests/UrdfParserTest.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/SdfAssetBuilder/SdfAssetBuilderSettings.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Improved asset path resolution logic and unit tests.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Add SDFormat importer hooks (o3de#453)

Add sensors SDF importer hooks

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* use PrimitiveAssets Gem to visualize primitives (o3de#485)

Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* Warehouse automation gem (o3de#440)

Created the warehouse automation gem, adjusted to review

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>

* Lidar component refactor (o3de#463)

Lidar Refactor

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>

* Added presentation of SDF messages (o3de#491)

* Added presentation of SDF messages

---------
Signed-off-by: Michał Pełka <michalpelka@gmail.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>

* Fix missing include.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Fix resolution of file:// prefix.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Change string to Path.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Changed PathResolver to get read in wholesale from setreg file.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback - changed file select dir to default to last-used file.
Also changes file select dir to match anything typed into the text box.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Partial revert from path back to string.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Second half of path to string revert.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Added all Path Resolver options to setreg for visibility.
Improved SetFindCallback to use the full set of path resolution options.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

---------

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>
Co-authored-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>
Co-authored-by: Michał Pełka <michal.pelka@robotec.ai>
Signed-off-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka pushed a commit to RobotecAI/o3de-extras that referenced this pull request Oct 2, 2023
Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka added a commit to RobotecAI/o3de-extras that referenced this pull request Oct 2, 2023
…remappings (o3de#505)

* First version of adding configurable prefix lookups and substitutions.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Tests/UrdfParserTest.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/SdfAssetBuilder/SdfAssetBuilderSettings.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Improved asset path resolution logic and unit tests.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Add SDFormat importer hooks (o3de#453)

Add sensors SDF importer hooks

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* use PrimitiveAssets Gem to visualize primitives (o3de#485)

Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* Warehouse automation gem (o3de#440)

Created the warehouse automation gem, adjusted to review

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>

* Lidar component refactor (o3de#463)

Lidar Refactor

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>

* Added presentation of SDF messages (o3de#491)

* Added presentation of SDF messages

---------
Signed-off-by: Michał Pełka <michalpelka@gmail.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>

* Fix missing include.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Fix resolution of file:// prefix.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Change string to Path.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Changed PathResolver to get read in wholesale from setreg file.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback - changed file select dir to default to last-used file.
Also changes file select dir to match anything typed into the text box.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Partial revert from path back to string.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Second half of path to string revert.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Added all Path Resolver options to setreg for visibility.
Improved SetFindCallback to use the full set of path resolution options.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

---------

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>
Co-authored-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>
Co-authored-by: Michał Pełka <michal.pelka@robotec.ai>
Signed-off-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka pushed a commit that referenced this pull request Oct 3, 2023
Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Michał Pełka <michal.pelka@robotec.ai>
michalpelka added a commit that referenced this pull request Oct 3, 2023
…remappings (#505)

* First version of adding configurable prefix lookups and substitutions.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Tests/UrdfParserTest.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/SdfAssetBuilder/SdfAssetBuilderSettings.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Improved asset path resolution logic and unit tests.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Add SDFormat importer hooks (#453)

Add sensors SDF importer hooks

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* use PrimitiveAssets Gem to visualize primitives (#485)

Use PrimitiveAssets Gem to visualize primitives

---------

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

* Warehouse automation gem (#440)

Created the warehouse automation gem, adjusted to review

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>

* Lidar component refactor (#463)

Lidar Refactor

---------

Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>

* Added presentation of SDF messages (#491)

* Added presentation of SDF messages

---------
Signed-off-by: Michał Pełka <michalpelka@gmail.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>

* Fix missing include.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Fix resolution of file:// prefix.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Change string to Path.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Changed PathResolver to get read in wholesale from setreg file.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback - changed file select dir to default to last-used file.
Also changes file select dir to match anything typed into the text box.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.h

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Update Gems/ROS2/Code/Source/RobotImporter/Utils/RobotImporterUtils.cpp

Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* PR feedback

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Partial revert from path back to string.

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Second half of path to string revert.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

* Added all Path Resolver options to setreg for visibility.
Improved SetFindCallback to use the full set of path resolution options.
Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>

---------

Signed-off-by: Mike Balfour <82224783+mbalfour-amzn@users.noreply.github.com>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Antoni Puch <antoni.puch@robotec.ai>
Signed-off-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: lumberyard-employee-dm <56135373+lumberyard-employee-dm@users.noreply.github.com>
Co-authored-by: Jan Hanca <134940295+jhanca-robotecai@users.noreply.github.com>
Co-authored-by: Antoni-Robotec <138497503+Antoni-Robotec@users.noreply.github.com>
Co-authored-by: Adam Dąbrowski <adam.dabrowski@robotec.ai>
Co-authored-by: Michał Pełka <michal.pelka@robotec.ai>
Signed-off-by: Michał Pełka <michal.pelka@robotec.ai>
@jhanca-robotecai jhanca-robotecai deleted the jh/import_fbx_primitives branch November 7, 2023 14:01
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.

URDF Importer - Visual primitives are created as debug components.
5 participants