Skip to content

Conversation

MaciejFilipow
Copy link
Collaborator

… RobotecGenericDecorationsAssets,

  • RobotecFenceAssets Gem: Bollard01, Modular Fences
  • RobotecSurveillanceCameras Gem: CCTVCamera01, CCTVCamera02
Fences

Checklist Bollard01:

  • All names are in English
  • Meshes, sub-meshes and textures following naming conventions
  • Z is up, X is forward
  • Flat hierarchy in model (no nested sub-meshes)
  • FBX lives next to “Materials/Textures” directory and all textures are in “Materials/Textures”, without additional extra directories
  • Pivot is set accordingly (different for steerable objects, and different for non steerable object)
  • Textures are max 4k x 4k
  • Transparent materials have alpha value set to ≠ 1.0
  • No unused materials/textures references are exported in FBX

Checklist CCTVCameras:

  • All names are in English
  • Meshes, sub-meshes and textures following naming conventions
  • Z is up, X is forward
  • Flat hierarchy in model (no nested sub-meshes)
  • FBX lives next to “Materials/Textures” directory and all textures are in “Materials/Textures”, without additional extra directories
  • Pivot is set accordingly (different for steerable objects, and different for non steerable object)
  • Textures are max 4k x 4k
  • Transparent materials have alpha value set to ≠ 1.0
  • No unused materials/textures references are exported in FBX

Checklist ModularFences:

  • All names are in English
  • Meshes, sub-meshes and textures following naming conventions
  • Z is up, X is forward
  • Flat hierarchy in model (no nested sub-meshes)
  • FBX lives next to “Materials/Textures” directory and all textures are in “Materials/Textures”, without additional extra directories
  • Pivot is set accordingly (different for steerable objects, and different for non steerable object)
  • Textures are max 4k x 4k
  • Transparent materials have alpha value set to ≠ 1.0
  • No unused materials/textures references are exported in FBX

… RobotecGenericDecorationsAssets,

+ RobotecFenceAssets Gem: Bollard01, Modular Fences
+ RobotecSurveillanceCameras Gem: CCTVCamera01, CCTVCamera02
@pawelbudziszewski pawelbudziszewski self-requested a review September 4, 2025 10:01
@pawelbudziszewski
Copy link
Member

pawelbudziszewski commented Sep 4, 2025

This PR is a derivative of RobotecAI/robotec-warehouse-assets#9

Copy link
Member

@pawelbudziszewski pawelbudziszewski left a comment

Choose a reason for hiding this comment

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

Everything looks and works fine. Some minor comments:

  • If the fire extinguisher is not our in-house model, please add info to the NOTICE file
  • Please bump RobotecGenericDecorationsAssets version to 2.0.0
  • Please make sure that all textures are connected to the materials (or remove the texture file)
  • Please move both CCTV cams so that the origin is in the mounting point
Screenshot from 2025-09-09 10-12-14
  • Please make the collider for the extinguisher pipe a bit smaller:
Screenshot from 2025-09-09 10-10-24

- Fix CCTV cams origin
- Fix CCTVs materials
- change RobotecGenericDecorationsAssets to 2.0.0
- Update fire extinguisher collider
@pawelbudziszewski
Copy link
Member

Everything is ok now.
One last request: change the gem name of RobotecSurveillanceCameras to RobotecSurveillanceCamerasAssets to be in line with other gems

@pawelbudziszewski pawelbudziszewski merged commit 40341a4 into main Sep 10, 2025
@pawelbudziszewski pawelbudziszewski deleted the mf/AddGems_Add3DAssets branch September 10, 2025 11:03
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.

2 participants