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

WarehouseAutomation Gem: incorrect prefabs #634

Open
jhanca-robotecai opened this issue Dec 15, 2023 · 2 comments
Open

WarehouseAutomation Gem: incorrect prefabs #634

jhanca-robotecai opened this issue Dec 15, 2023 · 2 comments
Labels
good-first-issue Issue ready for a new contributor according to the help wanted guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/minor Lowest priority. Work that may be scheduled sig/simulation Categorizes an issue or PR as relevant to SIG Simulation

Comments

@jhanca-robotecai
Copy link
Contributor

jhanca-robotecai commented Dec 15, 2023

Describe the bug
There are two incorrect assets in WarehouseAutomation Gem.

  1. Assets/Factory/EuroPallet.prefab
  2. Assets/Factory/TableUnderRobot.prefab

The first one does not load at all due to incorrect paths of mesh and material. When loaded manually, it is very big. The second one is rotated by -90 deg over x axis, which means it lies on a side.

Expected behavior
Prefabs are usable.

Actual behavior
Prefabs are not usable.

Screenshots/Video
image
image

Found in Branch
development: 0ec4ac5

Additional context
It should be double-checked if those resources are needed in this Gem -> it looks like a duplication of the functionality of WarehouseAssets Gem.

@jhanca-robotecai jhanca-robotecai added kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. good-first-issue Issue ready for a new contributor according to the help wanted guidelines. labels Dec 15, 2023
@lemonade-dm lemonade-dm added sig/simulation Categorizes an issue or PR as relevant to SIG Simulation and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Jan 3, 2024
@lemonade-dm
Copy link
Contributor

It appears the EuroPallet prefab has it's uniform scale set to 100 for some reason

@michalpelka michalpelka added priority/minor Lowest priority. Work that may be scheduled and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Jan 4, 2024
@michalpelka
Copy link
Contributor

Assigned minor priority. Please make sure that the changes introduced to adjust those issues are not affecting RosConDemo2023 and Project Templates. If so adjust another project accordingly with PRs to their development branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-first-issue Issue ready for a new contributor according to the help wanted guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/minor Lowest priority. Work that may be scheduled sig/simulation Categorizes an issue or PR as relevant to SIG Simulation
Projects
None yet
Development

No branches or pull requests

3 participants