Skip to content

Sm/remove unused or duplicated assets - #342

Open
Sebastian306 wants to merge 18 commits into
developmentfrom
sm/remove_unused_or_duplicated_assets
Open

Sm/remove unused or duplicated assets#342
Sebastian306 wants to merge 18 commits into
developmentfrom
sm/remove_unused_or_duplicated_assets

Conversation

@Sebastian306

@Sebastian306 Sebastian306 commented Jul 15, 2026

Copy link
Copy Markdown

What does this PR do?

Remove unused/duplicate assets already provided by gems.
TableUnderRobot.prefab, EuroPallet.prefab - provided by WarehouseAutomation
BeltTextures - project copy unused
PaintCan.fbx, PlasticBarrel.fbx, PlasticBucket.fbx, PlasticCanister.fbx - unused copies from RobotecSmallContainersAssets
Cage Walls - provided by RobotecWarehouseBarriersAndCagesAssets
ColisionBarierCornerType2 - unused copies from RobotecWarehouseBarriersAndCagesAssets

Move assets specific to this demo to gems

  1. robotec-warehouse-assets
    • Robot_camera_mount (renamed to CameraMount)
    • WheelRollMonitor
    • InspectionGate
    • Wrapper

After pushing changes on this gems, additional commit may be needed to switch subrepositories to newest version of main branch for this gems.

How was this PR tested?

Run demo.

@Sebastian306
Sebastian306 marked this pull request as ready for review July 20, 2026 08:46
Signed-off-by: Patryk Antosz <patryk.antosz@robotec.ai>
patrykantosz
patrykantosz previously approved these changes Jul 22, 2026

@jhanca-robotecai jhanca-robotecai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great work! Thank you for the effort!

I am flagging it with request changes, so we do not merge it accidentally. The submodule needs to be updated before we merge it, as we want to be sure we don't link it to the feature branch. People can force-push or remove the feature branch, making this release unstable. Let's wait for RobotecAI/robotec-warehouse-assets#37

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.

3 participants